KazydubB commented on a change in pull request #2092:
URL: https://github.com/apache/drill/pull/2092#discussion_r449480526



##########
File path: 
contrib/format-spss/src/test/java/org/apache/drill/exec/store/spss/TestSpssReader.java
##########
@@ -108,7 +108,7 @@ public void testStarQuery() throws Exception {
 
   @Test
   public void testExplicitQuery() throws Exception {
-    String sql = "SELECT ID, Urban, Urban_value FROM dfs.`spss/testdata.sav` 
WHERE d16=4";
+    String sql = "SELECT ID, Urban, Urban_value FROM dfs.`spss/testdata.sav` 
WHERE d16=4 LIMIT 5";

Review comment:
       Why not write another test method with `LIMIT` instead of editing the 
existing one without a `LIMIT`?




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


Reply via email to