-
Notifications
You must be signed in to change notification settings - Fork 179
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
deps: upgrade hbase2 to 2.3.6 #3429
Conversation
72ec6e1
to
7027ff4
Compare
2193b4e
to
e39d26a
Compare
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 after nits and questions addressed
<dependency> | ||
<groupId>net.bytebuddy</groupId> | ||
<artifactId>byte-buddy</artifactId> | ||
<version>1.12.6</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.
nit - I think this is used now in more than one place? Can we extract the version number to a property?
<google.bigtable.project.id>fake-project</google.bigtable.project.id> | ||
<google.bigtable.instance.id>fake-instance</google.bigtable.instance.id> | ||
<google.bigtable.test_env>bigtable</google.bigtable.test_env> |
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.
nit - can revert 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.
This was updated in https://github.com/googleapis/java-bigtable-hbase/pull/3452/files to use bigtable/minicluster instead of connection.impl.. I think I might have missed this pom file
<dependency> | ||
<groupId>org.slf4j</groupId> | ||
<artifactId>slf4j-api</artifactId> | ||
<version>1.7.30</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.
can we use the variable here?
...arent/bigtable-hbase/src/main/java/org/apache/hadoop/hbase/client/BigtableAsyncRegistry.java
Show resolved
Hide resolved
621a42c
to
0d4edb3
Compare
Port changes in #3293 and use bytebuddy to resolve hbase backward incompatible changes in
snapshotAsync
method and AsyncRegistery class.