Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-3362][SQL] bug in casewhen resolve #2245

Closed
wants to merge 4 commits into from

Conversation

adrian-wang
Copy link
Contributor

Current implementation will ignore else val type.

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have started for PR 2245 at commit eed35fc.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have started for PR 2245 at commit d7315b3.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have finished for PR 2245 at commit eed35fc.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public final class JavaKinesisWordCountASL
    • case class ColumnMetrics(

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have finished for PR 2245 at commit d7315b3.

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

@adrian-wang
Copy link
Contributor Author

The failure is in PageRankSuite, test it again, please.

@adrian-wang
Copy link
Contributor Author

and the first failure is in pyspark

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have started for PR 2245 at commit 83b536c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have finished for PR 2245 at commit 83b536c.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class BlockManagerMaster(
    • class AttributeMap[A](baseMap: Map[ExprId, (Attribute, A)])

@adrian-wang
Copy link
Contributor Author

Seems so unlucky to be trapped in different test suite in every run.
@marmbrus @rxin Can you give the patch a retest?

@rxin
Copy link
Contributor

rxin commented Sep 5, 2014

Jenkins is down right now ...

@adrian-wang
Copy link
Contributor Author

All right...

@marmbrus
Copy link
Contributor

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Sep 10, 2014

QA tests have started for PR 2245 at commit 3332f6e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 10, 2014

QA tests have finished for PR 2245 at commit 3332f6e.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Last(child: Expression) extends PartialAggregate with trees.UnaryNode[Expression]
    • case class LastFunction(expr: Expression, base: AggregateExpression) extends AggregateFunction
    • case class Abs(child: Expression) extends UnaryExpression

@adrian-wang
Copy link
Contributor Author

failure is in pyspark...

@marmbrus
Copy link
Contributor

@JoshRosen, here's an example of the pyspark failure.

@marmbrus
Copy link
Contributor

Jenkins, test this please.

@marmbrus
Copy link
Contributor

Actually, I'm just going to merge... Thanks!

@asfgit asfgit closed this in a028330 Sep 10, 2014
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