Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented Aug 22, 2019

What changes were proposed in this pull request?

This PR implements SparkGetCatalogsOperation for Thrift Server metadata completeness.

Why are the changes needed?

Thrift Server metadata completeness.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Unit test

@wangyum
Copy link
Member Author

wangyum commented Aug 22, 2019

cc @juliuszsompolski

@juliuszsompolski
Copy link
Contributor

LGTM! Thanks.
cc @gatorsmile

@SparkQA
Copy link

SparkQA commented Aug 22, 2019

Test build #109574 has finished for PR 25555 at commit ac8304e.

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

Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

Thanks! Merged to master.

@wangyum wangyum deleted the SPARK-28852 branch August 26, 2019 05:50
@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Aug 29, 2019

Hi, All. This seems to break JDK11 compilation (building with JDK11).

Could you take a look at this? Otherwise, we need to revert this first and make another PR to pass Jenkins with [test-hadoop3.2][test-java11].

Let me take a look at this more.

@dongjoon-hyun
Copy link
Member

Sorry, guys. It seems to be my environment issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants