Skip to content
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-1786] [BugFix] Refactored code to avoid null pointer exception while data loading #1543

Closed
wants to merge 1 commit into from

Conversation

geetikagupta16
Copy link
Contributor

Refactored code to add hadoop conf to AbstractDFS file constructor

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1333/

@ravipesala
Copy link
Contributor

SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1791/

@geetikagupta16
Copy link
Contributor Author

retest this please

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1336/

@geetikagupta16
Copy link
Contributor Author

@ravipesala Please review. This PR will resolve the current NullPointer exception for data loading. SDV test cases are failing due to the same issue.

@ravipesala
Copy link
Contributor

@jackylk @QiangCai Please check this PR. It seems PR #1539 makes lot of SDV test fails. Now current master fails in SDV because of this PR 1539 is merged

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1357/

@ravipesala
Copy link
Contributor

SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1811/

@ravipesala
Copy link
Contributor

PR 1550 fixed it, please close it.

@ravipesala
Copy link
Contributor

SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1817/

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1367/

@chenliang613
Copy link
Contributor

@geetikagupta16 thanks for your contribution, pr1550 already fixed this issue and merged, please close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants