arina-ielchiieva commented on a change in pull request #1912: DRILL-7324: Final 
set of "batch count" fixes
URL: https://github.com/apache/drill/pull/1912#discussion_r352345098
 
 

 ##########
 File path: 
exec/java-exec/src/test/java/org/apache/drill/exec/vector/complex/writer/TestJsonReader.java
 ##########
 @@ -81,6 +81,8 @@ public void testEmptyList() throws Exception {
 
   @Test
   public void schemaChange() throws Exception {
+    // TODO: What is this testing? Presumably, we are simply
 
 Review comment:
   I assume yes, it just tests that test pass, do we need this comment here 
though?

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to