[GitHub] carbondata pull request #2988: [CARBONDATA-3174] Fix trailing space issue wi...

[ajantha-bhat]

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

    https://github.com/apache/carbondata/pull/2988#discussion_r242157568
  
    --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/Field.java 
---
    @@ -55,7 +55,7 @@
        * @param type datatype of field, specified in strings.
        */
       public Field(String name, String type) {
    -    this.name = name;
    +    this.name = name.toLowerCase().trim();
    --- End diff --
    
    CarbonWriterBuilder.updateSchemaFields() is already converting to 
lowercase, just add trim in that method. No need to handle for each here.


---