Skip to content

Fixes sparkDriver port binding issue #2578

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.

Sign up for GitHub

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

Jump to bottom
Merged
farodin91 merged 1 commit into from
Apr 19, 2021
Merged

Fixes sparkDriver port binding issue #2578

farodin91 merged 1 commit into from
Apr 19, 2021

Conversation

@farodin91
Copy link
Contributor

@farodin91 farodin91 commented Apr 19, 2021 •
edited
Loading

Fixes #2577

For all changes:

  • Is there an issue associated with this PR? Is it referenced in the commit message?
  • Does your PR body contain #xyz where xyz is the issue number you are trying to resolve?
  • Has your PR been rebased against the latest commit within the target branch (typically master)?
  • Is your initial contribution a single, squashed commit?

@farodin91
Fixes sparkDriver port binding issue
7062442 
Fixes JanusGraph#2577

Signed-off-by: Jan Jansen <jan.jansen@gdata.de>
@farodin91 farodin91 requested a review from li-boxuan April 19, 2021 06:47
@farodin91 farodin91 added this to the Release v0.6.0 milestone Apr 19, 2021
@janusgraph-bot janusgraph-bot added the cla: external Externally-managed CLA label Apr 19, 2021
li-boxuan
li-boxuan approved these changes Apr 19, 2021
View reviewed changes
Copy link
Member

@li-boxuan li-boxuan left a comment

Choose a reason for hiding this comment

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

Do we need the same for hbase-read.properties which is used in HBaseInputFormatIT?

@farodin91
Copy link
Contributor Author

farodin91 commented Apr 19, 2021

Do we need the same for hbase-read.properties which is used in HBaseInputFormatIT?

I don't think we need this current version of hbase.

@farodin91 farodin91 merged commit c2f2d41 into JanusGraph:master Apr 19, 2021
@farodin91 farodin91 mentioned this pull request Apr 23, 2021
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@li-boxuan li-boxuan li-boxuan approved these changes

Assignees

No one assigned

Labels

cla: external Externally-managed CLA

Projects

None yet

Milestone

Release v0.6.0

Development

Successfully merging this pull request may close these issues.

GitHub Actions start to fail for CQLInputFormatIT on master

3 participants

@farodin91 @li-boxuan @janusgraph-bot