Skip to content

Conversation

@gatorsmile
Copy link
Member

What changes were proposed in this pull request?

Remove the AnalysisBarrier LogicalPlan node, which is useless now.

How was this patch tested?

N/A

@gatorsmile gatorsmile changed the title [SPARK-24865] Remove AnalysisBarrier LogicalPlan Node [SPARK-24865][FOLLOW-UP] Remove AnalysisBarrier LogicalPlan Node Aug 2, 2018
@SparkQA
Copy link

SparkQA commented Aug 2, 2018

Test build #93957 has finished for PR 21962 at commit 7f70aaa.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@gatorsmile
Copy link
Member Author

cc @rxin @cloud-fan

@cloud-fan
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Aug 2, 2018

Test build #93961 has finished for PR 21962 at commit 0135ba4.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Aug 2, 2018

Test build #93988 has finished for PR 21962 at commit 0135ba4.

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

@HyukjinKwon
Copy link
Member

Merged to master.

@asfgit asfgit closed this in 46110a5 Aug 2, 2018
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.

5 participants