Skip to content

Conversation

@liancheng
Copy link
Contributor

What changes were proposed in this pull request?

This PR is a follow-up of #13079. It replaces hasUnsupportedFeatures: Boolean in CatalogTable with unsupportedFeatures: Seq[String], which contains unsupported Hive features of the underlying Hive table. In this way, we can accurately report all unsupported Hive features in the exception message.

How was this patch tested?

Updated existing test case to check exception message.

@liancheng
Copy link
Contributor Author

cc @yhuai @andrewor14

@SparkQA
Copy link

SparkQA commented May 18, 2016

Test build #58781 has finished for PR 13173 at commit 3fb03af.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented May 18, 2016

Test build #58782 has finished for PR 13173 at commit 950cbe4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It sounds like that we are missing functionalities. Can we update the error message?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

@SparkQA
Copy link

SparkQA commented May 18, 2016

Test build #58802 has finished for PR 13173 at commit ca22d71.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@liancheng
Copy link
Contributor Author

retest this please

@SparkQA
Copy link

SparkQA commented May 19, 2016

Test build #58845 has finished for PR 13173 at commit ca22d71.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@liancheng liancheng force-pushed the spark-14346-follow-up branch from ca22d71 to 925e50a Compare May 19, 2016 12:27
@SparkQA
Copy link

SparkQA commented May 19, 2016

Test build #58865 has finished for PR 13173 at commit 925e50a.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor

Looks good. Merging into master 2.0.

@asfgit asfgit closed this in 6ac1c3a May 19, 2016
asfgit pushed a commit that referenced this pull request May 19, 2016
…LE output

## What changes were proposed in this pull request?

This PR is a follow-up of #13079. It replaces `hasUnsupportedFeatures: Boolean` in `CatalogTable` with `unsupportedFeatures: Seq[String]`, which contains unsupported Hive features of the underlying Hive table. In this way, we can accurately report all unsupported Hive features in the exception message.

## How was this patch tested?

Updated existing test case to check exception message.

Author: Cheng Lian <[email protected]>

Closes #13173 from liancheng/spark-14346-follow-up.

(cherry picked from commit 6ac1c3a)
Signed-off-by: Andrew Or <[email protected]>
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.

4 participants