Github user ajantha-bhat commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/3048#discussion_r245246228
  
    --- Diff: store/CSDK/test/main.cpp ---
    @@ -526,6 +526,71 @@ void testCarbonProperties(JNIEnv *env) {
         }
     }
     
    +bool testValidateBadRecordsActionWithImproperValue(JNIEnv *env, char 
*path) {
    --- End diff --
    
    Test case in this file is redundent. Because CarbonReaderTest.java already 
test same. 
    If SDK throws exception. CSDK will print it. NO need to duplicate test cases


---

Reply via email to