Skip to content

Conversation

liancheng
Copy link
Contributor

Class names of these two are just too similar.

@liancheng
Copy link
Contributor Author

test this please

@liancheng
Copy link
Contributor Author

ok to test

1 similar comment
@marmbrus
Copy link
Contributor

ok to test

@liancheng
Copy link
Contributor Author

test this please

@SparkQA
Copy link

SparkQA commented Sep 1, 2014

QA tests have started for PR 2189 at commit 8bb3b21.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 1, 2014

QA tests have finished for PR 2189 at commit 8bb3b21.

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

@marmbrus
Copy link
Contributor

marmbrus commented Sep 3, 2014

Thanks! I've merged this.

@asfgit asfgit closed this in 19d3e1e Sep 3, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
…ColumnStats

Class names of these two are just too similar.

Author: Cheng Lian <lian.cs.zju@gmail.com>

Closes apache#2189 from liancheng/column-metrics and squashes the following commits:

8bb3b21 [Cheng Lian] Renamed ColumnStat to ColumnMetrics to avoid confusion between ColumnStats
@liancheng liancheng deleted the column-metrics branch September 24, 2014 00:05
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