Skip to content

Conversation

@disq
Copy link
Member

@disq disq commented May 18, 2023

Neptune doesn't support list types (among other things) so we need to represent lists as strings.

The option defaults to false (assuming we have more Neptune users than Apache TinkerPop users it was worded as such)

Fixes #10839

@disq disq requested review from a team and amanenk and removed request for a team May 18, 2023 10:19
@disq disq requested a review from hermanschaaf May 18, 2023 10:19
@disq disq changed the title feat(gremlin): Add neptune_compatibility option feat(gremlin): Add complete_types option May 18, 2023
Copy link
Contributor

@candiduslynx candiduslynx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only 1 nit, otherwise good

@disq disq added the automerge Automatically merge once required checks pass label May 18, 2023
@kodiakhq kodiakhq bot merged commit 43166e1 into cloudquery:main May 18, 2023
@disq disq deleted the feat/gremlin-add-neptune-compatibility-option branch May 18, 2023 12:38
kodiakhq bot pushed a commit that referenced this pull request May 18, 2023
🤖 I have created a release *beep* *boop*
---


## [2.1.0](plugins-destination-gremlin-v2.0.2...plugins-destination-gremlin-v2.1.0) (2023-05-18)


### Features

* **deps:** Upgrade to Apache Arrow v13 (latest `cqmain`) ([#10605](#10605)) ([a55da3d](a55da3d))
* **gremlin:** Add `complete_types` option ([#10846](#10846)) ([43166e1](43166e1)), closes [#10839](#10839)
* **gremlin:** Upgrade to SDK v3 ([#10821](#10821)) ([27fc337](27fc337))


### Bug Fixes

* **deps:** Update module github.com/aws/aws-sdk-go-v2/config to v1.18.25 ([#10786](#10786)) ([caca1a4](caca1a4))
* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.0.8 ([#10798](#10798)) ([27ff430](27ff430))
* **gremlin:** Upgrade gremlingo to official 3.6.3 ([#10824](#10824)) ([656f2a2](656f2a2))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(Gremlin): Unsupported property value type: java.util.ArrayList

4 participants