You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2020/06/18 03:17:44 UTC
[incubator-pinot] branch master updated: (Minor) Remove the console
output in LiteralOnlyBrokerRequestTest (#5585)
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 9e0496b (Minor) Remove the console output in LiteralOnlyBrokerRequestTest (#5585)
9e0496b is described below
commit 9e0496bb27e0d111d781cb02ebb4ac1eeb4fb41a
Author: Xiaotian (Jackie) Jiang <17...@users.noreply.github.com>
AuthorDate: Wed Jun 17 20:17:30 2020 -0700
(Minor) Remove the console output in LiteralOnlyBrokerRequestTest (#5585)
---
.../broker/requesthandler/LiteralOnlyBrokerRequestTest.java | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/pinot-broker/src/test/java/org/apache/pinot/broker/requesthandler/LiteralOnlyBrokerRequestTest.java b/pinot-broker/src/test/java/org/apache/pinot/broker/requesthandler/LiteralOnlyBrokerRequestTest.java
index 0ca4983..2eebbb1 100644
--- a/pinot-broker/src/test/java/org/apache/pinot/broker/requesthandler/LiteralOnlyBrokerRequestTest.java
+++ b/pinot-broker/src/test/java/org/apache/pinot/broker/requesthandler/LiteralOnlyBrokerRequestTest.java
@@ -51,10 +51,9 @@ public class LiteralOnlyBrokerRequestTest {
@Test
public void testSelectStarBrokerRequestFromSQL() {
Assert.assertTrue(BaseBrokerRequestHandler.isLiteralOnlyQuery(SQL_COMPILER.compileToBrokerRequest("SELECT '*'")));
- Assert.assertTrue(BaseBrokerRequestHandler
- .isLiteralOnlyQuery(SQL_COMPILER.compileToBrokerRequest("SELECT '*' FROM myTable")));
- Assert.assertFalse(
- BaseBrokerRequestHandler.isLiteralOnlyQuery(SQL_COMPILER.compileToBrokerRequest("SELECT *")));
+ Assert.assertTrue(
+ BaseBrokerRequestHandler.isLiteralOnlyQuery(SQL_COMPILER.compileToBrokerRequest("SELECT '*' FROM myTable")));
+ Assert.assertFalse(BaseBrokerRequestHandler.isLiteralOnlyQuery(SQL_COMPILER.compileToBrokerRequest("SELECT *")));
Assert.assertFalse(
BaseBrokerRequestHandler.isLiteralOnlyQuery(SQL_COMPILER.compileToBrokerRequest("SELECT * FROM myTable")));
}
@@ -101,7 +100,6 @@ public class LiteralOnlyBrokerRequestTest {
RequestStatistics requestStats = new RequestStatistics();
BrokerResponseNative brokerResponse =
(BrokerResponseNative) requestHandler.handleRequest(request, null, requestStats);
- System.out.println("brokerResponse = " + brokerResponse.toJsonString());
Assert.assertEquals(brokerResponse.getResultTable().getDataSchema().getColumnName(0), String.format("%d", randNum));
Assert.assertEquals(brokerResponse.getResultTable().getDataSchema().getColumnDataType(0),
DataSchema.ColumnDataType.LONG);
@@ -128,7 +126,6 @@ public class LiteralOnlyBrokerRequestTest {
BrokerResponseNative brokerResponse =
(BrokerResponseNative) requestHandler.handleRequest(request, null, requestStats);
long currentTsMax = System.currentTimeMillis();
- System.out.println("brokerResponse = " + brokerResponse.toJsonString());
Assert.assertEquals(brokerResponse.getResultTable().getDataSchema().getColumnName(0), "currentTs");
Assert.assertEquals(brokerResponse.getResultTable().getDataSchema().getColumnDataType(0),
DataSchema.ColumnDataType.LONG);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org