-
Notifications
You must be signed in to change notification settings - Fork 647
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
HBase: version 1.4.9 #1654
HBase: version 1.4.9 #1654
Conversation
We need to be careful when upgrading versions to keep binary compatibility and might need to defer it to a non-patch release. |
project/Dependencies.scala
Outdated
Seq( | ||
libraryDependencies ++= Seq( | ||
// for some reason version 2.2.3U1 started to get picked which was not accepted by Whitesource) | ||
"com.sun.xml.bind" % "jaxb-impl" % "2.2.3-1", // CDDL + GPLv2 | ||
// TODO: remove direct dependency ^^ when updating from these very old versions |
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.
Can the direct jaxb-impl
dependency go away now?
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.
I removed the jaxb-impl
dependency. HBase tests are OK, but UnixDomainSocketSpec is failing somehow.
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.
No worries about UnixDomainSocket.
Does it still depend on jaxb-impl
? What version?
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.
HBase 1.4.9 includes jaxb-impl 2.2.3-1
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.
We upgraded some dependencies in test scope already. Can you rebase this and resolve the conflicts, please? |
I rebased this branch on current master. |
Failure was #1801 |
Thank you for this upgrade! |
Purpose
This change is to upgrade Apache HBase to 1.4.9, which is the current stable version.
Changes
This PR changed an HBase docker image to run tests using HBase 1.4.