huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-743699192
This has been merged and close this PR.
This is an automated message from the Apache Git Service.
To respond
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-721930030
Update addressing Duo and Stack's comments. If there is anything I need to
take care of, please let me know.
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-721889440
> Landing this on master is proposed by me as this PR is not related to the
server side changes. It can be used in our current code base, without the
changes in HBASE-18070.
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-721441167
Upload a new one to address Nick's comments which did not show up at the
front page (embedded in the patch).
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-721374482
@Apache9 @saintstack @ndimiduk Latest patch uploaded which addresses Duo's
latest comments, please help to review, thanks.
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-720249209
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-719066991
Pushed a separate patch which adds metaReplicaId to RegionLocations. It is
being used to hold from which meta replica locations come from. It adds 4 bytes
(int) to
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-719052727
Update to address some of comments from Stack.
This is an automated message from the Apache Git Service.
To
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-718332088
I had a run for the failed test case TestRSGroupsAdmin2, it passed. I assume
this test is flakey.
This is an
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-718273513
@saintstack @Apache9 I pushed updates based on your comments, could you take
a look to see if all comments are addressed? Thanks.
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-716774965
Uploaded a new patch.
This is an automated message from the Apache Git Service.
To respond to the message,
huaxiangsun commented on pull request #2584:
URL: https://github.com/apache/hbase/pull/2584#issuecomment-716751088
Changing the newly added test to Parameterized one, will load a new patch
shortly.
This is an automated
12 matches
Mail list logo