[GitHub] carbondata pull request #2710: [2875]two different threads overwriting the s...

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #2710: [2875]two different threads overwriting the s...

qiuchenjian-2
GitHub user shardul-cr7 opened a pull request:

    https://github.com/apache/carbondata/pull/2710

    [2875]two different threads overwriting the same carbondatafile

    Problem : Two different threads are overwriting the same carbondata file during creation of external table.
   
    Solution: Chances of two threads concurrently loading same carbondatafile is reduced by changing the timestamp attached in .carbondata file from millisecond to nanosecond.So chances of collision of different threads having the same file name is reduced.
   
    Be sure to do all of the following checklist to help us incorporate
    your contribution quickly and easily:
   
     - [ ] Any interfaces changed?
     
     - [ ] Any backward compatibility impacted?
     
     - [ ] Document update required?
   
     - [x] Testing done
             Done Manually
           
     - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
   


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shardul-cr7/carbondata master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/carbondata/pull/2710.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2710
   
----
commit 61520a3d0bacfbcbed5a2b5ac300f08cf9b36bb4
Author: shardul-cr7 <shardulsingh22@...>
Date:   2018-09-11T12:51:09Z

    chances of two different threads overwriting the same carbondatafile is reduced

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2710: [2875]two different threads overwriting the same car...

qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2710
 
    Can one of the admins verify this patch?


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #2710: [2875]two different threads overwriting the same car...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2710
 
    Can one of the admins verify this patch?


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #2710: [2875]two different threads overwriting the s...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user shardul-cr7 closed the pull request at:

    https://github.com/apache/carbondata/pull/2710


---