[GitHub] carbondata pull request #2773: [CARBONDATA-2978] Fixed JVM crash issue when ...

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

[GitHub] carbondata pull request #2773: [CARBONDATA-2978] Fixed JVM crash issue when ...

qiuchenjian-2
GitHub user ravipesala opened a pull request:

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

    [CARBONDATA-2978] Fixed JVM crash issue when insert into carbon table from other carbon table

    Problem:
    When data is inserted from one carbon to other carbon table and unsafe load and query is enabled then JVM crash is happening.
    Reason: When insert happens from one carbon table another table it uses same task and thread so it gets the same taskid and at the unsafe manager tries to release all memory acquired by the task even though load happens on the task.
    Solution:
    Check the listeners and ignore cache clearing.
   
    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?
   
     - [ ] Testing done
            Please provide details on
            - Whether new unit test cases have been added or why no new tests are required?
            - How it is tested? Please attach test report.
            - Is it a performance related change? Please attach the performance test report.
            - Any additional information to help reviewers in testing this change.
           
     - [ ] 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/ravipesala/incubator-carbondata insert-crash-issue

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

    https://github.com/apache/carbondata/pull/2773.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 #2773
   
----
commit 0f8602d6404e7a9c974086f4106a98d8dfb62ae9
Author: ravipesala <ravi.pesala@...>
Date:   2018-09-26T17:34:59Z

    Fixed JVM crash issue when insert into carbon table from other carbon table

----


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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2773
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/512/



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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/776/



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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    Build Success with Spark 2.3.1, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/8843/



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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    retest this please


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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/628/



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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/820/



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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    Build Success with Spark 2.3.1, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/8889/



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

[GitHub] carbondata issue #2773: [CARBONDATA-2978] Fixed JVM crash issue when insert ...

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

    https://github.com/apache/carbondata/pull/2773
 
    LGTM


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

[GitHub] carbondata pull request #2773: [CARBONDATA-2978] Fixed JVM crash issue when ...

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

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


---