You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by GitBox <gi...@apache.org> on 2020/08/07 05:20:14 UTC

[GitHub] [carbondata] ajantha-bhat edited a comment on pull request #3773: [CARBONDATA-3830]Presto array columns read support

ajantha-bhat edited a comment on pull request #3773:
URL: https://github.com/apache/carbondata/pull/3773#issuecomment-669844507


   @akkio-97 : update limitations and TODO clearly
   a. with local dictionary arrays cannot be read now
   b. arrays with other complex type is not supported yet
   c. currently, array is row by row filling, not really vector processing.  can use offset vector-like ORC
   https://github.com/prestosql/presto/blob/master/presto-orc/src/main/java/io/prestosql/orc/reader/ListColumnReader.java
   
   I also feel arrayStreamRader and some interface need to cleaned up [I will do it with struct support]


----------------------------------------------------------------
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