[GitHub] [carbondata] akkio-97 edited a comment on pull request #4115: [CARBONDATA-4163] Support adding of single-level complex columns(array/struct)

[GitBox <gi...@apache.org>]

akkio-97 edited a comment on pull request #4115:
URL: https://github.com/apache/carbondata/pull/4115#issuecomment-819673737


   > @akkio-97 For alter add columns, currently we support a property to provide default value in alter add ddl, like "DEFAULT.VALUE.column_name=somevalue". Please check and handle this scenario also.
   
   1) This is a part of the query flow. Because only when the select command is fired the data assigned will be inserted into the row.
   2) The data which we assign as default is not parsed through queryUtil flow and is just a byte array. This data is not encoded so existing parsers and decoders cannot be used here to extract actual data and insert into the row.
   3) We need to first write a new parser and then create ArrayQueryType and PrimitiveQueryType. After which insertion can be done.
   4) This I think can be handled in a separate PR


-- 
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:
users@infra.apache.org