Skip to content

Commit 64517e4

Browse files
committed
fix checkstyle
1 parent efe5583 commit 64517e4

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed

hbase-client/src/main/java/org/apache/hadoop/hbase/replication/ReplicationPeerConfig.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
* A configuration for the replication peer cluster.
3434
*/
3535
@InterfaceAudience.Public
36-
public class ReplicationPeerConfig {
36+
public final class ReplicationPeerConfig {
3737

3838
private String clusterKey;
3939
private String replicationEndpointImpl;

hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestMasterReplication.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -634,7 +634,8 @@ private void addPeer(String id, int masterClusterNumber, int slaveClusterNumber,
634634
Admin admin = conn.getAdmin()) {
635635
admin.addReplicationPeer(
636636
id,
637-
ReplicationPeerConfig.newBuilder().setClusterKey(utilities[slaveClusterNumber].getClusterKey())
637+
ReplicationPeerConfig.newBuilder()
638+
.setClusterKey(utilities[slaveClusterNumber].getClusterKey())
638639
.setReplicateAllUserTables(false)
639640
.setTableCFsMap(ReplicationPeerConfigUtil.parseTableCFsFromConfig(tableCfs)).build());
640641
}

hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestReplicationEndpoint.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -141,10 +141,11 @@ public String explainFailure() throws Exception {
141141
@Test
142142
public void testCustomReplicationEndpoint() throws Exception {
143143
// test installing a custom replication endpoint other than the default one.
144-
hbaseAdmin.addReplicationPeer("testCustomReplicationEndpoint", ReplicationPeerConfig.newBuilder()
145-
.setClusterKey(ZKConfig.getZooKeeperClusterKey(CONF1))
146-
.setReplicationEndpointImpl(ReplicationEndpointForTest.class.getName())
147-
.build());
144+
hbaseAdmin.addReplicationPeer("testCustomReplicationEndpoint",
145+
ReplicationPeerConfig.newBuilder()
146+
.setClusterKey(ZKConfig.getZooKeeperClusterKey(CONF1))
147+
.setReplicationEndpointImpl(ReplicationEndpointForTest.class.getName())
148+
.build());
148149

149150
// check whether the class has been constructed and started
150151
Waiter.waitFor(CONF1, 60000, new Waiter.Predicate<Exception>() {
@@ -188,8 +189,7 @@ public void testReplicationEndpointReturnsFalseOnReplicate() throws Exception {
188189
ReplicationPeerConfig.newBuilder()
189190
.setClusterKey(ZKConfig.getZooKeeperClusterKey(CONF1))
190191
.setReplicationEndpointImpl(ReplicationEndpointReturningFalse.class.getName())
191-
.build()
192-
);
192+
.build());
193193
// This test is flakey and then there is so much stuff flying around in here its, hard to
194194
// debug. Peer needs to be up for the edit to make it across. This wait on
195195
// peer count seems to be a hack that has us not progress till peer is up.
@@ -242,8 +242,8 @@ public void testInterClusterReplication() throws Exception {
242242
hbaseAdmin.addReplicationPeer(id,
243243
ReplicationPeerConfig.newBuilder()
244244
.setClusterKey(ZKConfig.getZooKeeperClusterKey(CONF2))
245-
.setReplicationEndpointImpl(InterClusterReplicationEndpointForTest.class.getName()).build()
246-
);
245+
.setReplicationEndpointImpl(InterClusterReplicationEndpointForTest.class.getName())
246+
.build());
247247

248248
final int numEdits = totEdits;
249249
Waiter.waitFor(CONF1, 30000, new Waiter.ExplainingPredicate<Exception>() {

0 commit comments

Comments
 (0)