You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@carbondata.apache.org by GitBox <gi...@apache.org> on 2021/09/07 04:59:04 UTC

[GitHub] [carbondata] ShreelekhyaG opened a new pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

ShreelekhyaG opened a new pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214


    ### Why is this PR needed?
    1. Insert/load fails after alter add complex column if table contains long string columns.
    2. create index on array of complex column (map/struct) throws null pointer exception instead of correct error message.
    
    ### What changes were proposed in this PR?
   1. The datatypes array and data row are of different order leading to `ClassCastException`. Made changes to add newly added complex columns after the long string columns and other dimensions.
   2. For complex columns, SI creation on only array of primitive types is allowed. Check if the child column is of complex type and throw an exception.
       
    ### Does this PR introduce any user interface change?
    - No
   
    ### Is any new testcase added?
    - No, edited existing testcases.
   
       
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

Posted by GitBox <gi...@apache.org>.
CarbonDataQA2 commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914051729


   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5909/
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

Posted by GitBox <gi...@apache.org>.
CarbonDataQA2 commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914448270


   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5910/
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

Posted by GitBox <gi...@apache.org>.
CarbonDataQA2 commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914437831


   Build Success with Spark 3.1, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_3.1/312/
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] ajantha-bhat commented on pull request #4214: [CARBONDATA-4282] Fix issues with table having complex columns related to long string, SI, local dictionary

Posted by GitBox <gi...@apache.org>.
ajantha-bhat commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914965092


   LGTM


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

Posted by GitBox <gi...@apache.org>.
CarbonDataQA2 commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914054952


   Build Success with Spark 3.1, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_3.1/311/
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

Posted by GitBox <gi...@apache.org>.
CarbonDataQA2 commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914454575


   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/4166/
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] asfgit closed pull request #4214: [CARBONDATA-4282] Fix issues with table having complex columns related to long string, SI, local dictionary

Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214


   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4214: [WIP] Fix issues with table having complex columns related to long string, SI.

Posted by GitBox <gi...@apache.org>.
CarbonDataQA2 commented on pull request #4214:
URL: https://github.com/apache/carbondata/pull/4214#issuecomment-914056160


   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/4165/
   


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

To unsubscribe, e-mail: dev-unsubscribe@carbondata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org