Skip to content

[SQL] [Minor] added check analysis rule for union node - #6174

Closed
scwf wants to merge 2 commits into
apache:masterfrom
scwf:union
Closed

[SQL] [Minor] added check analysis rule for union node#6174
scwf wants to merge 2 commits into
apache:masterfrom
scwf:union

Conversation

@scwf

@scwfscwf commented May 15, 2015

Copy link
Copy Markdown
Contributor

No description provided.

@AmplabJenkins

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

Copy link
Copy Markdown

Merged build started.

@SparkQA

Copy link
Copy Markdown

Test build #32778 has started for PR 6174 at commit 6541ec4.

@SparkQA

Copy link
Copy Markdown

Test build #32778 has finished for PR 6174 at commit 6541ec4.

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

@AmplabJenkins

Copy link
Copy Markdown

Merged build finished. Test FAILed.

@AmplabJenkins

Copy link
Copy Markdown

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32778/
Test FAILed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It this logic already in Union.resolved?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

aha, yes it is .

@AmplabJenkins

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

Copy link
Copy Markdown

Merged build started.

@SparkQA

Copy link
Copy Markdown

Test build #32782 has started for PR 6174 at commit cf1475a.

@SparkQA

Copy link
Copy Markdown

Test build #32782 has finished for PR 6174 at commit cf1475a.

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

@AmplabJenkins

Copy link
Copy Markdown

Merged build finished. Test FAILed.

@AmplabJenkins

Copy link
Copy Markdown

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32782/
Test FAILed.

@scwf

scwf commented May 15, 2015

Copy link
Copy Markdown
ContributorAuthor

seems in hive it allow union's children have diff outputsize
such as

explain FROM (select 'tst1' as key, cast(count(1) as string) as value from src s1
UNION ALL select s2.key as key, s2.value as value from src s2) unionsrc
INSERT OVERWRITE TABLE DEST1 SELECT unionsrc.key, COUNT(DISTINCT SUBSTR(unionsrc.value,5)) GROUP BY unionsrc.key
INSERT OVERWRITE TABLE DEST2 SELECT unionsrc.key, unionsrc.value, COUNT(DISTINCT SUBSTR(unionsrc.value,5)) GROUP BY unionsrc.key, unionsrc.value;

Sign up for freeto 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

@scwf@AmplabJenkins@SparkQA@cloud-fan