-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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-27061 two phase bulkload is broken when SFT is in use. #4462
Conversation
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
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.
Nice catch @ss77892 ! I think this variable naming is misleading. Since you are already here, can you also rename this regionDir
attribute to walRegionDir
as to avoid such future mistakes?
Also, I noticed this fix is also required in master
branch, so in these cases, we typically open PR to master branch instead, then once it's merged, we cherry-pick it to the lower branches. Would you mind target this PR to master
branch, instead?
I've opened a PR to the master branch. Also I've renamed regionDir to regionWalDir. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
I had merged the other PR on master. Will close this one and cherry-pick that commit into lower branches. |
To reproduce the bug I've used hbase with hbase root dir pointing to s3 bucket and WAL dir pointing to HDFS. 2 phase bulkload was used:
Phase1 (Completed successfully):
hbase org.apache.hadoop.hbase.mapreduce.ImportTsv -Dimporttsv.separator=',' -Dimporttsv.bulk.output=output -Dimporttsv.columns=HBASE_ROW_KEY,name,department employees /tmp/test.csv
Phase2 (Failed with IllegalArgumentException on the RS side):
hbase org.apache.hadoop.hbase.mapreduce.LoadIncrementalHFiles output employees
With the provided changes Phase2 was completed successfully.