You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by anubhav100 <gi...@git.apache.org> on 2018/03/06 06:41:42 UTC
[GitHub] carbondata pull request #2034: [CARBONDATA-2288] Exception is Masked Inside ...
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/2034
[CARBONDATA-2288] Exception is Masked Inside StandardPartitionTableQueryTestCase
**Problem**: Inside This test case exception from both spark version is different and it is masked
test("Creation of partition table should fail if the colname in table schema and partition column is same even if both are case sensitive"){
intercept[Exception]
{ sql("CREATE TABLE uniqdata_char2(name char,id int) partitioned by (NAME char)stored by 'carbondata' ") }
}
for spark 2.1 exception message is
Operation not allowed: Partition columns should not be specified in the schema: name")
but for spark 2.2 it is
DataType char is not supported"
**Reason**: for different exception in different spark version is
spark 2.1.0 supports creating table even if char data type is given with out digits
scala> spark.sql("create table id(id char)");
18/02/23 14:12:23 WARN HiveMetaStore: Location: file:/home/anubhav/Documents/phatak/spark-2.1/bin/spark-warehouse/id specified for non-external table:id
res0: org.apache.spark.sql.DataFrame = []
but not in spark 2.2.1
scala> spark.sql("create table id(id char)");
18/02/23 14:22:05 WARN ObjectStore: Failed to get database global_temp, returning NoSuchObjectException
org.apache.spark.sql.catalyst.parser.ParseException:
DataType char is not supported.(line 1, pos 19)
**Solution**: assert the different exception for different spark version when char data type does not have digit
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [ ] Any interfaces changed?
- [ ] Any backward compatibility impacted?
- [ ] Document update required?
- [ ] Testing done
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.
- [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/anubhav100/incubator-carbondata CARBONDATA-2288
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/carbondata/pull/2034.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 #2034
----
commit b454d3eb732c423e88d9af49c6a4535c9d121110
Author: anubhav100 <an...@...>
Date: 2018-03-06T06:38:48Z
Fixed Exception is Masked Inside StandardPartitionTableQueryTestCase
----
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] [Test] Exception is Masked Inside ...
Posted by jackylk <gi...@git.apache.org>.
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/2034
LGTM
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] [Test] Exception is Masked Inside ...
Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2034
SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3843/
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] [Test] Exception is Masked Inside ...
Posted by anubhav100 <gi...@git.apache.org>.
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/2034
retest sdv please
---
[GitHub] carbondata pull request #2034: [CARBONDATA-2288] [Test] Exception is Masked ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/2034
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] [Test] Exception is Masked Inside ...
Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2034
SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3878/
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] Exception is Masked Inside Standar...
Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2034
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/4082/
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] [Test] Exception is Masked Inside ...
Posted by xubo245 <gi...@git.apache.org>.
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2034
retest sdv please
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] Exception is Masked Inside Standar...
Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2034
Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/2837/
---
[GitHub] carbondata issue #2034: [CARBONDATA-2288] [Test] Exception is Masked Inside ...
Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2034
SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3781/
---