You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by akashrn5 <gi...@git.apache.org> on 2017/11/15 08:58:59 UTC

[GitHub] carbondata pull request #1500: [WIP]Remove spark broadcast for gettting hado...

GitHub user akashrn5 opened a pull request:

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

    [WIP]Remove spark broadcast for gettting hadoop configurations

    why?
    with the new conf set in hadoop conf, after broadcast conf will not get reflected, hence these changes 
    Be sure to do all of the following checklist to help us incorporate 
    your contribution quickly and easily:
    
     - [*] Any interfaces changed?
     NONE
     - [*] Any backward compatibility impacted?
     NONE
     - [ ] Document update required?
    NONE
     - [*] Testing done
           test cases are not required, SDV will take care
           
     - [*] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. 
    NONE
    


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

    $ git pull https://github.com/akashrn5/incubator-carbondata conf

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

    https://github.com/apache/carbondata/pull/1500.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 #1500
    
----
commit e86bf0809dd1270126b99f5c61f92c84ba5a2ed8
Author: akashrn5 <ak...@gmail.com>
Date:   2017-10-27T12:41:03Z

    remove spark broadcast for gettting hadoop configurations

----


---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1150/



---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1173/



---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1149/



---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by QiangCai <gi...@git.apache.org>.
Github user QiangCai commented on the issue:

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



---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by akashrn5 <gi...@git.apache.org>.
Github user akashrn5 commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    @QiangCai please review


---

[GitHub] carbondata pull request #1500: [CARBONDATA-1717]Remove spark broadcast for g...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1147/



---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by akashrn5 <gi...@git.apache.org>.
Github user akashrn5 commented on the issue:

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


---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by QiangCai <gi...@git.apache.org>.
Github user QiangCai commented on the issue:

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


---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by akashrn5 <gi...@git.apache.org>.
Github user akashrn5 commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    @ravipesala @jackylk please review


---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by QiangCai <gi...@git.apache.org>.
Github user QiangCai commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    please fix CI issue


---

[GitHub] carbondata issue #1500: [CARBONDATA-1717]Remove spark broadcast for gettting...

Posted by akashrn5 <gi...@git.apache.org>.
Github user akashrn5 commented on the issue:

    https://github.com/apache/carbondata/pull/1500
  
    retest sdv pleaase


---