-
Notifications
You must be signed in to change notification settings - Fork 788
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rework remote console #2611
rework remote console #2611
Conversation
@@ -39,6 +39,12 @@ limitations under the License. | |||
<artifactId>tinkergraph-gremlin</artifactId> | |||
<version>${project.version}</version> | |||
</dependency> | |||
<dependency> | |||
<groupId>com.fasterxml.jackson.core</groupId> | |||
<artifactId>jackson-databind</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this necessary? We purposely put jackson in gremlin-shaded to avoid doing things like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it doesn't work without this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what doesn't work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aws sdk used for sigv4 require jackson-databind
cluster = Cluster.build("localhost").port(8182).auth(sigv4("us-west2")).create()
g = traversal().withRemote(DriverRemoteConnection.using(cluster, "g"))
leads to
java.lang.NoClassDefFoundError: com/fasterxml/jackson/databind/ObjectMapper
gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/Settings.java
Outdated
Show resolved
Hide resolved
gremlin-console/src/main/groovy/org/apache/tinkerpop/gremlin/console/Console.groovy
Outdated
Show resolved
Hide resolved
gremlin-console/src/main/groovy/org/apache/tinkerpop/gremlin/console/Mediator.groovy
Outdated
Show resolved
Hide resolved
gremlin-console/src/main/java/org/apache/tinkerpop/gremlin/console/jsr223/ConnectionHelper.java
Outdated
Show resolved
Hide resolved
gremlin-console/src/test/groovy/org/apache/tinkerpop/gremlin/console/GremlinGroovyshTest.groovy
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.