This is an automated email from the ASF dual-hosted git repository. zhangduo pushed a commit to branch branch-3 in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-3 by this push: new f6f41c467e6 HBASE-28512 Update error prone to 2.26.1 (#5843) f6f41c467e6 is described below commit f6f41c467e631c011581a0096b669ef3f820289e Author: Duo Zhang <zhang...@apache.org> AuthorDate: Fri Apr 26 13:57:42 2024 +0800 HBASE-28512 Update error prone to 2.26.1 (#5843) Signed-off-by: Xin Sun <ddu...@gmail.com> --- .../src/test/java/org/apache/hadoop/hbase/http/TestHtmlQuoting.java | 3 ++- .../test/java/org/apache/hadoop/hbase/regionserver/TestRegionInfo.java | 2 +- .../org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java | 2 +- pom.xml | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/hbase-http/src/test/java/org/apache/hadoop/hbase/http/TestHtmlQuoting.java b/hbase-http/src/test/java/org/apache/hadoop/hbase/http/TestHtmlQuoting.java index e5835fd65b6..0fe46194083 100644 --- a/hbase-http/src/test/java/org/apache/hadoop/hbase/http/TestHtmlQuoting.java +++ b/hbase-http/src/test/java/org/apache/hadoop/hbase/http/TestHtmlQuoting.java @@ -20,6 +20,7 @@ package org.apache.hadoop.hbase.http; import static org.junit.Assert.assertArrayEquals; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNull; import static org.junit.Assert.assertTrue; import javax.servlet.http.HttpServletRequest; @@ -94,7 +95,7 @@ public class TestHtmlQuoting { quoter.getParameterValues("x")); Mockito.doReturn(null).when(mockReq).getParameterValues("x"); - assertArrayEquals("Test that missing parameters dont cause NPE for array", null, + assertNull("Test that missing parameters dont cause NPE for array", quoter.getParameterValues("x")); } } diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionInfo.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionInfo.java index de177ec3ace..60fe39ecc77 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionInfo.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionInfo.java @@ -320,7 +320,7 @@ public class TestRegionInfo { b = RegionInfoBuilder.newBuilder(t).setStartKey(midway).build(); assertTrue(a.compareTo(b) < 0); assertTrue(b.compareTo(a) > 0); - assertEquals(a, a); + assertTrue(a.equals(a)); assertEquals(0, a.compareTo(a)); a = RegionInfoBuilder.newBuilder(t).setStartKey(Bytes.toBytes("a")) .setEndKey(Bytes.toBytes("d")).build(); diff --git a/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java b/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java index 9e9e1554aac..810bb87baa9 100644 --- a/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java +++ b/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift2/TestThriftHBaseServiceHandler.java @@ -536,7 +536,7 @@ public class TestThriftHBaseServiceHandler { get = new TGet(wrap(rowName)); result = handler.get(table, get); - assertArrayEquals(null, result.getRow()); + assertNull(result.getRow()); assertEquals(0, result.getColumnValuesSize()); } diff --git a/pom.xml b/pom.xml index 3bb3b4ae929..9b5d6c7fa84 100644 --- a/pom.xml +++ b/pom.xml @@ -880,7 +880,7 @@ --> <checkstyle.version>8.29</checkstyle.version> <exec.maven.version>3.1.0</exec.maven.version> - <error-prone.version>2.16</error-prone.version> + <error-prone.version>2.26.1</error-prone.version> <jamon.plugin.version>2.4.2</jamon.plugin.version> <lifecycle.mapping.version>1.0.0</lifecycle.mapping.version> <maven.antrun.version>1.8</maven.antrun.version>