This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new a09305d5854 HBASE-28297 IntegrationTestImportTsv fails with 
ArrayIndexOfOutBounds (#5612)
a09305d5854 is described below

commit a09305d5854fc98300426271fad3b53a69d2ae71
Author: Duo Zhang <zhang...@apache.org>
AuthorDate: Wed Jan 10 11:13:09 2024 +0800

    HBASE-28297 IntegrationTestImportTsv fails with ArrayIndexOfOutBounds 
(#5612)
    
    Signed-off-by: Nihal Jain <nihalj...@apache.org>
---
 .../org/apache/hadoop/hbase/mapreduce/IntegrationTestImportTsv.java   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git 
a/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestImportTsv.java
 
b/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestImportTsv.java
index 35db989dd69..2e9ff827936 100644
--- 
a/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestImportTsv.java
+++ 
b/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestImportTsv.java
@@ -60,6 +60,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import org.apache.hbase.thirdparty.com.google.common.base.Splitter;
+import org.apache.hbase.thirdparty.com.google.common.base.Strings;
 
 /**
  * Validate ImportTsv + BulkLoadFiles on a distributed cluster.
@@ -85,6 +86,9 @@ public class IntegrationTestImportTsv extends Configured 
implements Tool {
       {
         byte[] family = Bytes.toBytes("d");
         for (String line : Splitter.on('\n').split(simple_tsv)) {
+          if (Strings.isNullOrEmpty(line)) {
+            continue;
+          }
           String[] row = line.split("\t");
           byte[] key = Bytes.toBytes(row[0]);
           long ts = Long.parseLong(row[1]);

Reply via email to