[GitHub] [carbondata] Klaus-xjp opened a new pull request #3929: [CARBONDATA-3991] Fix the modified time failed problem

[GitBox]

Klaus-xjp opened a new pull request #3929:
URL: https://github.com/apache/carbondata/pull/3929


   Why is this PR needed?
   If there are some update or create mv in S3 and Alluxio files ,those operations do not take effect. And the other tenant can't find the changes in this operation and cause a data consistency problem.
   
   What changes were proposed in this PR?
   In this PR, we set two function to fix this problem.
   1. Create a new file for S3 and Alluxio to set a tag to update.
   2. Set a switch case like function to select different scenario in update or create.
   
   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]

[GitBox]

CarbonDataQA1 commented on pull request #3929:
URL: https://github.com/apache/carbondata/pull/3929#issuecomment-693124094


   Can one of the admins verify this patch?


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

[GitBox]

QiangCai commented on pull request #3929:
URL: https://github.com/apache/carbondata/pull/3929#issuecomment-693131254


   if the old file exists, how does it work?


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

[GitBox]

QiangCai commented on pull request #3929:
URL: https://github.com/apache/carbondata/pull/3929#issuecomment-693131340


   retest this please


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

[GitBox]

Klaus-xjp closed pull request #3929:
URL: https://github.com/apache/carbondata/pull/3929


   


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

[GitBox]

CarbonDataQA1 commented on pull request #3929:
URL: https://github.com/apache/carbondata/pull/3929#issuecomment-693160345


   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4090/
   


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