GitHub user kushalsaha opened a pull request:
https://github.com/apache/carbondata/pull/1457 [CARBONDATA-1627] one job failed among 100 job while performing selec⦠â¦t operation with 100 different thread Be sure to do all of the following checklist to help us incorporate your contribution quickly and easily: - [ ] Any interfaces changed? NO - [ ] Any backward compatibility impacted? NO - [ ] Document update required? No - [ ] 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. => this issue is resolved when we launch 100 job through jmeter and one job was getting failed because multiple thread would try to initialize the variable even tough the call is made through lazy, but still in multiple thread cases multiple thread tried to initialize variable, to avoid this added synchronized block so that only one thread only can initialize the variable and latter for other thread there's no chance of accessing that block. - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. No large changes You can merge this pull request into a Git repository by running: $ git pull https://github.com/kushalsaha/carbondata CARBONDATA-1627 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/carbondata/pull/1457.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 #1457 ---- commit ff80dbe6fe09dd8f1487e5fe99d5ce20aff77c3e Author: kushalsaha <[hidden email]> Date: 2017-10-27T12:47:50Z [CARBONDATA-1627] one job failed among 100 job while performing select operation with 100 different thread ---- --- |
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 Can one of the admins verify this patch? --- |
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1457 Can one of the admins verify this patch? --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 add to whitelist --- |
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1457 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/829/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1463/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 @kushalsaha please rebase the code --- |
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1457 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/960/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 @kushalsaha Please squash to single commit, I cannot rebase our PR --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1575/ --- |
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1457 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/998/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1615/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 @kushalsaha please rebase the code --- |
In reply to this post by qiuchenjian-2
Github user kushalsaha commented on the issue:
https://github.com/apache/carbondata/pull/1457 @ravipesala conflicts resolved please merge --- |
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1457 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1104/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1705/ --- |
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1457 LGTM --- |
In reply to this post by qiuchenjian-2
|
Free forum by Nabble | Edit this page |