[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Build Failed  with Spark 2.3.2, Please check CI 
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10330/



---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Build Failed with Spark 2.2.1, Please check CI 
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2281/



---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Build Failed  with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2076/



---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread chenliang613
Github user chenliang613 commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
@runzhliu  please correct the PR title.


---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Build Failed with Spark 2.2.1, Please check CI 
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2280/



---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Build Failed  with Spark 2.3.2, Please check CI 
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10329/



---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Build Failed  with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2075/



---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread qiuchenjian
Github user qiuchenjian commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
@xubo245 OK, i'll try


---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread xubo245
Github user xubo245 commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
add to whitelist


---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread xubo245
Github user xubo245 commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
@qiuchenjian I tried it before, but there are some limit, not easy to 
support it. If you have better way to support it, you can raise a PR to 
implement it.


---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread qiuchenjian
Github user qiuchenjian commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
@xuchuanyin   @xubo245 @QiangCai  @jackylk  Can we enable checkstyle rule 
of avoiding unused imports,i think it's necessary to keep good code style and 
reduce the dependency of a class on the number of jar packages,  the rule may 
be is   


---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

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


---


[GitHub] carbondata issue #3036: [CARBONDATA-3208]Remove unused parameters and import...

2018-12-28 Thread runzhliu
Github user runzhliu commented on the issue:

https://github.com/apache/carbondata/pull/3036
  
Hi @xubo245 , can you please take a look?


---