Skip to content

Conversation

@sun-rui
Copy link
Contributor

@sun-rui sun-rui commented Nov 18, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Nov 18, 2015

Test build #46207 has finished for PR 9804 at commit a199d79.

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

@sun-rui
Copy link
Contributor Author

sun-rui commented Nov 30, 2015

rebased to master

@SparkQA
Copy link

SparkQA commented Nov 30, 2015

Test build #46863 has finished for PR 9804 at commit b4b0a61.

  • This patch fails R style tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * case class Count(children: Seq[Expression]) extends DeclarativeAggregate\n

@SparkQA
Copy link

SparkQA commented Nov 30, 2015

Test build #46869 has finished for PR 9804 at commit f5add6e.

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

@SparkQA
Copy link

SparkQA commented Nov 30, 2015

Test build #46877 has finished for PR 9804 at commit b253205.

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

@shivaram
Copy link
Contributor

shivaram commented Dec 5, 2015

LGTM. Thanks @sun-rui -- Sorry for the delay in reviewing this. Merging this to master and branch-1.6

@asfgit asfgit closed this in c8d0e16 Dec 5, 2015
asfgit pushed a commit that referenced this pull request Dec 5, 2015
…s in SparkR.

Author: Sun Rui <[email protected]>

Closes #9804 from sun-rui/SPARK-11774.

(cherry picked from commit c8d0e16)
Signed-off-by: Shivaram Venkataraman <[email protected]>
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.

3 participants