[GitHub] [carbondata] akashrn5 commented on pull request #3947: [CARBONDATA-4003] Improve IUD Concurrency

2020-09-29 Thread GitBox


akashrn5 commented on pull request #3947:
URL: https://github.com/apache/carbondata/pull/3947#issuecomment-700534709


   @Kejian-Li i can just see the test file changes and no source file changes 
for the issue which you described. So im confused what exactly or how exactly 
the problem is solved. can you please explain?



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




[GitHub] [carbondata] akashrn5 commented on pull request #3947: [CARBONDATA-4003] Improve IUD Concurrency

2020-09-29 Thread GitBox


akashrn5 commented on pull request #3947:
URL: https://github.com/apache/carbondata/pull/3947#issuecomment-700503471


   @Kejian-Li from the PR description, you have mentioned the problem is we 
cannot run the update to table when the insert is in progress, which is valid, 
that's because we want to maintain the data consistency. But you suggest we 
remove that check to allow the update. How do you plan to manage the data 
consistency when the insert in progress? You have not mentioned clearly in the 
description and its not clear from changes also. 
   
   As @Zhangshunyu asked same doubt in above comments, there is no clear reply. 
Can you please explain the solution you have given here?



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




[GitHub] [carbondata] akashrn5 commented on pull request #3947: [CARBONDATA-4003] Improve IUD Concurrency

2020-09-28 Thread GitBox


akashrn5 commented on pull request #3947:
URL: https://github.com/apache/carbondata/pull/3947#issuecomment-700489664


   > please check mail list:
   > 
http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/DISCUSSION-Parallel-compaction-and-update-td100338.html
   > 
   > How do we process the parallel compaction(not only loading) and update?
   
   I involved in that solution derivation, But its recommended and community's 
suggestion to start working on it after segment refactoring.



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