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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
