Skip to content

Conversation

@jtoka
Copy link
Contributor

@jtoka jtoka commented Apr 11, 2017

What changes were proposed in this pull request?

Update count distinct error message for streaming datasets/dataframes to match current behavior. These aggregations are not yet supported, regardless of whether the dataset/dataframe is aggregated.

Update count distinct error message for streaming datasets/dataframes to match current behavior. These aggregations are not yet supported.
@srowen
Copy link
Member

srowen commented Apr 11, 2017

CC @tdas to confirm that's correct

@tdas
Copy link
Contributor

tdas commented Apr 11, 2017

That's correct. LGTM.

@tdas
Copy link
Contributor

tdas commented Apr 11, 2017

jenkins test this

@SparkQA
Copy link

SparkQA commented Apr 12, 2017

Test build #3660 has finished for PR 17609 at commit a4d34c5.

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

asfgit pushed a commit that referenced this pull request Apr 12, 2017
## What changes were proposed in this pull request?
Update count distinct error message for streaming datasets/dataframes to match current behavior. These aggregations are not yet supported, regardless of whether the dataset/dataframe is aggregated.

Author: jtoka <[email protected]>

Closes #17609 from jtoka/master.

(cherry picked from commit 2e1fd46)
Signed-off-by: Sean Owen <[email protected]>
@srowen
Copy link
Member

srowen commented Apr 12, 2017

Merged to master/2.1

@asfgit asfgit closed this in 2e1fd46 Apr 12, 2017
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
## What changes were proposed in this pull request?
Update count distinct error message for streaming datasets/dataframes to match current behavior. These aggregations are not yet supported, regardless of whether the dataset/dataframe is aggregated.

Author: jtoka <[email protected]>

Closes apache#17609 from jtoka/master.
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