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-3230] Add alter test case for datasource #3024
Conversation
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1941/ |
Build Success with Spark 2.3.2, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/10194/ |
Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2150/ |
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1950/ |
Build Success with Spark 2.3.2, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/10203/ |
@xubo245 can you more tests related to drop column, rename column , change datatype of column also here. And also for rename table as well. |
@ravipesala OK |
Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2225/ |
…et and carbon 1.add column 2.drop column 3.rename column 4.change datatype of column
55adf73
to
53619cd
Compare
@ravipesala add test case in this PR: 1.add column Add test case for carbon and parquet. |
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2169/ |
Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2384/ |
Build Success with Spark 2.3.2, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/10424/ |
@KanakaKumar @ravipesala Please review it. |
LGTM |
[CARBONDATA-3230] Add ALTER test case with datasource for using parquet and carbon 1.add column. => carbon and parquet don't support, limit from Spark 2.drop column => carbon doesn't support in sql, limit from Spark, but using DF is ok; parquet use DF is ok, but sql doesn't support iy. 3.rename column =》 carbon and parquet support it. 4.change datatype of column => carbon parquet doesn't support, limit from Spark, spark only support change comment This closes #3024
[CARBONDATA-3230] Add ALTER test case with datasource for using parquet and carbon 1.add column. => carbon and parquet don't support, limit from Spark 2.drop column => carbon doesn't support in sql, limit from Spark, but using DF is ok; parquet use DF is ok, but sql doesn't support iy. 3.rename column =》 carbon and parquet support it. 4.change datatype of column => carbon parquet doesn't support, limit from Spark, spark only support change comment This closes apache#3024
[CARBONDATA-3230] Add ALTER test case with datasource for using parquet and carbon
Add test case for carbon and parquet.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
No
No
No
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.
added
No