Skip to content

Commit

Permalink
reformat ClusterTest.java (#12531)
Browse files Browse the repository at this point in the history
  • Loading branch information
sullis authored Mar 2, 2024
1 parent 954f5d7 commit 7b71bab
Showing 1 changed file with 4 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,7 @@ protected void startBrokers(int numBrokers)
_brokerPorts = new ArrayList<>();
for (int i = 0; i < numBrokers; i++) {
BaseBrokerStarter brokerStarter = startOneBroker(i);
assertEquals(
brokerStarter.getConfig().getProperty(
CommonConstants.CONFIG_OF_TIMEZONE, ""),
"UTC"
);
assertEquals(brokerStarter.getConfig().getProperty(CommonConstants.CONFIG_OF_TIMEZONE, ""), "UTC");
assertEquals(System.getProperty("user.timezone"), "UTC");

_brokerStarters.add(brokerStarter);
Expand Down Expand Up @@ -256,8 +252,7 @@ protected PinotConfiguration getServerConf(int serverId) {
serverConf.setProperty(Server.CONFIG_OF_ADMIN_API_PORT, _serverAdminApiPort);
_serverNettyPort = NetUtils.findOpenPort(Helix.DEFAULT_SERVER_NETTY_PORT + new Random().nextInt(10000) + serverId);
serverConf.setProperty(Helix.KEY_OF_SERVER_NETTY_PORT, _serverNettyPort);
_serverGrpcPort =
NetUtils.findOpenPort(Server.DEFAULT_GRPC_PORT + new Random().nextInt(10000) + serverId);
_serverGrpcPort = NetUtils.findOpenPort(Server.DEFAULT_GRPC_PORT + new Random().nextInt(10000) + serverId);
serverConf.setProperty(Server.CONFIG_OF_GRPC_PORT, _serverGrpcPort);

// Thread time measurement is disabled by default, enable it in integration tests.
Expand Down Expand Up @@ -622,17 +617,14 @@ public void assertNoError(JsonNode response) {
@DataProvider(name = "systemColumns")
public Object[][] systemColumns() {
return new Object[][]{
{"$docId"},
{"$hostName"},
{"$segmentName"}
{"$docId"}, {"$hostName"}, {"$segmentName"}
};
}

@DataProvider(name = "useBothQueryEngines")
public Object[][] useBothQueryEngines() {
return new Object[][]{
{false},
{true}
{false}, {true}
};
}

Expand Down

0 comments on commit 7b71bab

Please sign in to comment.