Skip to content

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