Skip to content

Commit 3b54675

Browse files
committed
checkstyle fixes
1 parent 059a647 commit 3b54675

File tree

2 files changed

+20
-9
lines changed

2 files changed

+20
-9
lines changed

phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -2183,7 +2183,8 @@ private MetaDataMutationResult metaDataCoprocessorExec(String tableName, byte[]
21832183
long startTime = EnvironmentEdgeManager.currentTimeMillis();
21842184
while (true) {
21852185
if (retried) {
2186-
connection.getRegionLocator(SchemaUtil.getPhysicalName(systemTableName, this.getProps()))
2186+
connection.getRegionLocator(SchemaUtil.getPhysicalName(
2187+
systemTableName, this.getProps()))
21872188
.getRegionLocation(tableKey, true);
21882189
}
21892190

phoenix-core-client/src/main/java/org/apache/phoenix/schema/MetaDataClient.java

+18-8
Original file line numberDiff line numberDiff line change
@@ -6524,15 +6524,20 @@ public MutationState changePermissions(ChangePermsStatement changePermsStatement
65246524
return new MutationState(0, 0, connection);
65256525
}
65266526

6527-
private void changePermsOnSchema(org.apache.hadoop.hbase.client.Connection hConnection, ChangePermsStatement changePermsStatement) throws Throwable {
6527+
private void changePermsOnSchema(org.apache.hadoop.hbase.client.Connection hConnection,
6528+
ChangePermsStatement changePermsStatement) throws Throwable {
65286529
if (changePermsStatement.isGrantStatement()) {
6529-
AccessControlClient.grant(hConnection, changePermsStatement.getSchemaName(), changePermsStatement.getName(), changePermsStatement.getPermsList());
6530+
AccessControlClient.grant(hConnection, changePermsStatement.getSchemaName(),
6531+
changePermsStatement.getName(), changePermsStatement.getPermsList());
65306532
} else {
6531-
AccessControlClient.revoke(hConnection, changePermsStatement.getSchemaName(), changePermsStatement.getName(), Permission.Action.values());
6533+
AccessControlClient.revoke(hConnection, changePermsStatement.getSchemaName(),
6534+
changePermsStatement.getName(), Permission.Action.values());
65326535
}
65336536
}
65346537

6535-
private void changePermsOnTables(org.apache.hadoop.hbase.client.Connection hConnection, Admin admin, ChangePermsStatement changePermsStatement, PTable inputTable) throws Throwable {
6538+
private void changePermsOnTables(org.apache.hadoop.hbase.client.Connection hConnection,
6539+
Admin admin, ChangePermsStatement changePermsStatement,
6540+
PTable inputTable) throws Throwable {
65366541

65376542
org.apache.hadoop.hbase.TableName tableName = SchemaUtil.getPhysicalTableName
65386543
(inputTable.getPhysicalName().getBytes(), inputTable.isNamespaceMapped());
@@ -6587,7 +6592,9 @@ private void changePermsOnTables(org.apache.hadoop.hbase.client.Connection hConn
65876592
}
65886593
}
65896594

6590-
private void changePermsOnTable(org.apache.hadoop.hbase.client.Connection hConnection, ChangePermsStatement changePermsStatement, org.apache.hadoop.hbase.TableName tableName)
6595+
private void changePermsOnTable(org.apache.hadoop.hbase.client.Connection hConnection,
6596+
ChangePermsStatement changePermsStatement,
6597+
org.apache.hadoop.hbase.TableName tableName)
65916598
throws Throwable {
65926599
if (changePermsStatement.isGrantStatement()) {
65936600
AccessControlClient.grant(hConnection, tableName, changePermsStatement.getName(),
@@ -6598,12 +6605,15 @@ private void changePermsOnTable(org.apache.hadoop.hbase.client.Connection hConne
65986605
}
65996606
}
66006607

6601-
private void changePermsOnUser(org.apache.hadoop.hbase.client.Connection hConnection, ChangePermsStatement changePermsStatement)
6608+
private void changePermsOnUser(org.apache.hadoop.hbase.client.Connection hConnection,
6609+
ChangePermsStatement changePermsStatement)
66026610
throws Throwable {
66036611
if (changePermsStatement.isGrantStatement()) {
6604-
AccessControlClient.grant(hConnection, changePermsStatement.getName(), changePermsStatement.getPermsList());
6612+
AccessControlClient.grant(hConnection, changePermsStatement.getName(),
6613+
changePermsStatement.getPermsList());
66056614
} else {
6606-
AccessControlClient.revoke(hConnection, changePermsStatement.getName(), Permission.Action.values());
6615+
AccessControlClient.revoke(hConnection, changePermsStatement.getName(),
6616+
Permission.Action.values());
66076617
}
66086618
}
66096619
}

0 commit comments

Comments
 (0)