Skip to content

Commit

Permalink
checkstyle fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
szucsvillo committed Dec 9, 2024
1 parent b20447f commit 059a647
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -677,7 +677,8 @@ public static PhoenixConnection getRebuildIndexConnection(Configuration config)
public static boolean tableRegionsOnline(Configuration conf, PTable table) {
try (Connection hcon = ConnectionFactory.createConnection(conf)) {
Admin admin = hcon.getAdmin();
List<RegionInfo> regionInfos = admin.getRegions(TableName.valueOf(table.getPhysicalName().getBytes()));
List<RegionInfo> regionInfos = admin.getRegions(TableName.valueOf(
table.getPhysicalName().getBytes()));
// This makes Number of Regions RPC calls sequentially.
// For large tables this can be slow.
for (RegionInfo regionInfo : regionInfos) {
Expand All @@ -692,7 +693,8 @@ public static boolean tableRegionsOnline(Configuration conf, PTable table) {
// very hard (needs reflection, or a bridge class in the same package),
// and it's not necessary for checking the RS liveness
} catch (IOException e) {
LOGGER.debug("Cannot get region " + regionInfo.getEncodedName() + " info due to error:" + e);
LOGGER.debug("Cannot get region " + regionInfo.getEncodedName()
+ " info due to error:" + e);
return false;
}
}
Expand Down

0 comments on commit 059a647

Please sign in to comment.