cgivre commented on a change in pull request #2200:
URL: https://github.com/apache/drill/pull/2200#discussion_r635231706
##########
File path:
contrib/storage-kafka/src/test/java/org/apache/drill/exec/store/kafka/KafkaFilterPushdownTest.java
##########
@@ -419,13 +419,13 @@ public void testNoPushdownOfOffsetWithNonMetadataField()
throws Exception {
final int expectedRowCount = 30;
final String queryString =
String.format(TestQueryConstants.QUERY_TEMPLATE_AND_OR_PATTERN_2,
- TestQueryConstants.JSON_PUSHDOWN_TOPIC, predicate1, predicate2,
predicate3);
+ TestQueryConstants.JSON_PUSHDOWN_TOPIC, predicate1, predicate2,
predicate3);
runKafkaSQLVerifyCount(queryString,expectedRowCount);
queryBuilder()
- .sql(queryString)
- .jsonPlanMatcher()
- .include(String.format(EXPECTED_PATTERN, expectedRowCountInPlan))
- .match();
+ .sql(queryString)
+ .jsonPlanMatcher()
+ .include(String.format(EXPECTED_PATTERN, expectedRowCountInPlan))
+ .match();
}
-}
+}
Review comment:
@vvysotskyi
I put the version of this file from `master` back in this PR, so this
**should** be what was originally there. I didn't modify this file at all, so
I'm not sure why the spacing got all messed up in the first place. I hope
that's ok.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
