Skip to content
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

Update to the latest janusgraph and gremlin #2755

Merged
merged 5 commits into from
Sep 13, 2021
Merged

Conversation

lmsurpre
Copy link
Member

The previous version included versions of jackson-databind and netty that had known vulnerabilities.

Signed-off-by: Lee Surprenant lmsurpre@us.ibm.com

The previous version included versions of jackson-databind and netty that had known vulnerabilities.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
The previous version included versions of jackson-databind and netty that had known vulnerabilities.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
lmsurpre and others added 3 commits September 13, 2021 10:25
The previous version included versions of jackson-databind and netty that had known vulnerabilities.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>
Passing the vertices won't work with gremlin 3.5.x due to
https://tinkerpop.apache.org/docs/3.5.1/upgrade/#_anonymous_child_traversals

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Copy link
Collaborator

@JohnTimm JohnTimm left a comment

Choose a reason for hiding this comment

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

LGTM

@lmsurpre lmsurpre merged commit 44c3dce into main Sep 13, 2021
@lmsurpre lmsurpre deleted the update-janusgraph branch September 13, 2021 18:58
lmsurpre added a commit that referenced this pull request Sep 17, 2021
I accidentally used a placeholder "myconfig.properties" instead of the
propFileName argument when I switched over to apache commons config 2.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
lmsurpre added a commit that referenced this pull request Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants