Skip to content

Conversation

@tdas
Copy link
Contributor

@tdas tdas commented Jul 17, 2017

What changes were proposed in this pull request?

Implementation may expose both timing as well as size metrics. This PR enables that.

@zsxwing
Copy link
Member

zsxwing commented Jul 17, 2017

LGTM pending tests

@SparkQA
Copy link

SparkQA commented Jul 18, 2017

Test build #79690 has finished for PR 18661 at commit 351c207.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • trait StateStoreCustomMetric
  • case class StateStoreCustomSizeMetric(name: String, desc: String) extends StateStoreCustomMetric
  • case class StateStoreCustomTimingMetric(name: String, desc: String) extends StateStoreCustomMetric

@tdas
Copy link
Contributor Author

tdas commented Jul 18, 2017

Merging to master.

@asfgit asfgit closed this in e9faae1 Jul 18, 2017
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