Skip to content

Commit

Permalink
[SPARK-32889][SQL][TESTS][FOLLOWUP] Skip special column names test in…
Browse files Browse the repository at this point in the history
… Hive 1.2

### What changes were proposed in this pull request?

This PR is a followup of SPARK-32889 in order to ignore the special column names test in `hive-1.2` profile.

### Why are the changes needed?

Hive 1.2 is too old to support special column names because it doesn't use Apache ORC. This will recover our `hive-1.2` Jenkins job.
- https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test%20(Dashboard)/job/spark-master-test-sbt-hadoop-2.7-hive-1.2/
- https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test%20(Dashboard)/job/spark-master-test-maven-hadoop-2.7-hive-1.2/

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the test with Hive 1.2 profile.

Closes #29867 from dongjoon-hyun/SPARK-32889-2.

Authored-by: Dongjoon Hyun <dhyun@apple.com>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
  • Loading branch information
dongjoon-hyun committed Sep 24, 2020
1 parent 8ccfbc1 commit d7aa3b5
Showing 1 changed file with 1 addition and 0 deletions.
Expand Up @@ -2242,6 +2242,7 @@ abstract class SQLQuerySuiteBase extends QueryTest with SQLTestUtils with TestHi
}

test("SPARK-32889: ORC table column name supports special characters") {
assume(HiveUtils.isHive23)
// " " "," is not allowed.
Seq("$", ";", "{", "}", "(", ")", "\n", "\t", "=").foreach { name =>
val source = "ORC"
Expand Down

0 comments on commit d7aa3b5

Please sign in to comment.