Skip to content

[SPARK-9592] [SQL] Fix Last function implemented based on AggregateExpression1.#8172

Closed
yhuai wants to merge 2 commits into
apache:masterfrom
yhuai:lastFix
Closed

[SPARK-9592] [SQL] Fix Last function implemented based on AggregateExpression1.#8172
yhuai wants to merge 2 commits into
apache:masterfrom
yhuai:lastFix

Conversation

@yhuai

@yhuai yhuai commented Aug 13, 2015

Copy link
Copy Markdown
Contributor

https://issues.apache.org/jira/browse/SPARK-9592
#8113 has the fundamental fix. But, if we want to minimize the number of changed lines, we can go with this one. Then, in 1.6, we merge #8113.

@yhuai yhuai changed the title [SPARK-9592] Fix Last function implemented based on AggregateExpression1. [SPARK-9592] [SQL] Fix Last function implemented based on AggregateExpression1. Aug 13, 2015
@SparkQA

SparkQA commented Aug 13, 2015

Copy link
Copy Markdown

Test build #40784 has finished for PR 8172 at commit b28c42a.

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

asfgit pushed a commit that referenced this pull request Aug 17, 2015
…pression1.

https://issues.apache.org/jira/browse/SPARK-9592

#8113 has the fundamental fix. But, if we want to minimize the number of changed lines, we can go with this one. Then, in 1.6, we merge #8113.

Author: Yin Huai <yhuai@databricks.com>

Closes #8172 from yhuai/lastFix and squashes the following commits:

b28c42a [Yin Huai] Regression test.
af87086 [Yin Huai] Fix last.

(cherry picked from commit 772e7c1)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 772e7c1 Aug 17, 2015
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.

2 participants