Skip to content

Conversation

@beliefer
Copy link
Contributor

@beliefer beliefer commented Jun 17, 2021

What changes were proposed in this pull request?

#32513 added the case class CommandResult in package org.apache.spark.sql.expression. It is not suitable, so this PR move CommandResult from org.apache.spark.sql.expression to org.apache.spark.sql.catalyst.plans.logical.

Why are the changes needed?

Make CommandResult in suitable package.

Does this PR introduce any user-facing change?

'No'.

How was this patch tested?

No need.

@github-actions github-actions bot added the SQL label Jun 17, 2021
@SparkQA
Copy link

SparkQA commented Jun 17, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44434/

@SparkQA
Copy link

SparkQA commented Jun 17, 2021

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44434/

@HyukjinKwon
Copy link
Member

@beliefer, can you rebase this please?

@SparkQA
Copy link

SparkQA commented Jun 17, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44445/

@SparkQA
Copy link

SparkQA commented Jun 17, 2021

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44445/

@SparkQA
Copy link

SparkQA commented Jun 17, 2021

Test build #139917 has finished for PR 32942 at commit 4e8c762.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • case class MakeDTInterval(

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Thank you, @beliefer and @HyukjinKwon .
Merged to master for Apache Spark 3.2.0.

@beliefer
Copy link
Contributor Author

@HyukjinKwon @dongjoon-hyun Thanks a lot!

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.

4 participants