[GitHub] carbondata pull request #2901: [CARBONDATA-3081] Fixed NPE for boolean type ...

[manishgupta88]

Github user manishgupta88 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2901#discussion_r230816465
  
    --- Diff: 
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonReaderTest.java ---
    @@ -1844,4 +1844,53 @@ public void testVectorReader() {
         }
       }
     
    +  @Test
    +  public void testReadingNullValues() {
    +    String path = "./testWriteFiles";
    +    try {
    +      FileUtils.deleteDirectory(new File(path));
    +
    +      Field[] fields = new Field[2];
    +      fields[0] = new Field("stringField", DataTypes.STRING);
    +      fields[1] = new Field("shortField", DataTypes.BOOLEAN);
    --- End diff --
    
    Rename `shortField` to `booleanField`


---