[GitHub] [carbondata] xuchuanyin edited a comment on issue #3188: [CARBONDATA-3336] Support configurable decode for loading binary data, support base64 and Hex decode.

[GitBox]

xuchuanyin edited a comment on issue #3188: [CARBONDATA-3336] Support configurable decode for loading binary data, support base64 and Hex decode.
URL: https://github.com/apache/carbondata/pull/3188#issuecomment-487063622
 
 
   CarbonData will only use the `binary decoder` to parse/convert the values in the input records and it won't affect the final store, thus modification of query process is not needed, right?
   
   Another related function needed may be supporting `dataframe.write` with `binary decoder` option.
   
   At last, I do suggest using a decoder hierarchy classes to deal with the decoders. Each decoder will have a decode method and return the decoded value. Just treat it as a simple compressor and follow the way of the implementation of compressors.

----------------------------------------------------------------
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:
[hidden email]


With regards,
Apache Git Services