kgeisz commented on code in PR #7730:
URL: https://github.com/apache/hbase/pull/7730#discussion_r4275241669
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/tool/BulkLoadHFilesTool.java:
##########
@@ -660,21 +660,21 @@ private int getRegionIndex(List<Pair<byte[], byte[]>>
startEndKeys, byte[] key)
private void checkRegionIndexValid(int idx, List<Pair<byte[], byte[]>>
startEndKeys,
TableName tableName) throws IOException {
if (idx < 0) {
- throw new IOException("The first region info for table " + tableName
- + " can't be found in hbase:meta.Please use hbck tool to fix it
first.");
+ throw new IOException("The first region info for table " + tableName + "
can't be found in "
+ + "hbase:meta. Please use hbck tool to fix it first.");
} else if (
(idx == startEndKeys.size() - 1)
&& !Bytes.equals(startEndKeys.get(idx).getSecond(),
HConstants.EMPTY_BYTE_ARRAY)
) {
- throw new IOException("The last region info for table " + tableName
- + " can't be found in hbase:meta.Please use hbck tool to fix it
first.");
+ throw new IOException("The last region info for table " + tableName + "
can't be found in "
+ + "hbase:meta. Please use hbck tool to fix it first.");
Review Comment:
This comment has been addressed. I can't mark it as "resolved" because the
comment is outdated.
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/tool/BulkLoadHFilesTool.java:
##########
@@ -660,21 +660,21 @@ private int getRegionIndex(List<Pair<byte[], byte[]>>
startEndKeys, byte[] key)
private void checkRegionIndexValid(int idx, List<Pair<byte[], byte[]>>
startEndKeys,
TableName tableName) throws IOException {
if (idx < 0) {
- throw new IOException("The first region info for table " + tableName
- + " can't be found in hbase:meta.Please use hbck tool to fix it
first.");
+ throw new IOException("The first region info for table " + tableName + "
can't be found in "
+ + "hbase:meta. Please use hbck tool to fix it first.");
Review Comment:
This comment has been addressed. I can't mark it as "resolved" because the
comment is outdated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]