Login  Register

[GitHub] [carbondata] Karan980 opened a new pull request #3970: [CARBONDATA-4007] Fix multiple issues in SDK_IUD

Posted by GitBox on Oct 07, 2020; 7:03am
URL: http://apache-carbondata-dev-mailing-list-archive.168.s1.nabble.com/GitHub-carbondata-Karan980-opened-a-new-pull-request-3970-CARBONDATA-4007-Fix-multiple-issues-in-SDKD-tp101429.html


Karan980 opened a new pull request #3970:
URL: https://github.com/apache/carbondata/pull/3970


    ### Why is this PR needed?
   Fix multiple issues occurred in SDK_IUD.
   a) TupleId always have linux file separator independent of the system.
   b) Filtered rows array size gives ArrayOutOfBound exception if number of deleted rows is greater than 4096.
   c) While writing the data of date column type during update operation gives bad record exception as dates are converted to Integer during read.
   
    ### What changes were proposed in this PR?
   a) Changed the tupleId file separator to linux file separator.
   b) Change the filtered rows size to default column page rows size.
   c) Converted the integer back to date type before writing the data during update operation.
   
     ### Does this PR introduce any user interface change?
    - No
   
    ### Is any new testcase added?
    - Yes
   
       
   


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