-
Notifications
You must be signed in to change notification settings - Fork 703
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CARBONDATA-1717]Remove spark broadcast for gettting hadoop configurations #1500
Conversation
@QiangCai please review |
Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1147/ |
@ravipesala @jackylk please review |
Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1149/ |
retest this please |
Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1150/ |
please fix CI issue |
retest this please |
retest sdv pleaase |
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1173/ |
LGTM |
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