[GitHub] spark issue #19775: [SPARK-22343][core] Add support for publishing Spark met...

2018-02-08 Thread matyix
Github user matyix commented on the issue: https://github.com/apache/spark/pull/19775 Hello @erikerlandson @felixcheung @jerryshao - any feedback on this PR? Shall I close it and not worry about this being merged upstream anymore? We've been using this in production for the l

[GitHub] spark issue #19775: [SPARK-22343][core] Add support for publishing Spark met...

2018-03-08 Thread matyix
Github user matyix commented on the issue: https://github.com/apache/spark/pull/19775 @GaalDornick @erikerlandson @jerryshao @felixcheung et all We gave up this - we have made the requested changes several times and I am not willing to put more time on this and get in the

[GitHub] spark pull request #19775: [SPARK-22343][core] Add support for publishing Sp...

2018-03-08 Thread matyix
Github user matyix closed the pull request at: https://github.com/apache/spark/pull/19775 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #19775: [SPARK-22343][core] Add support for publishing Spark met...

2018-03-10 Thread matyix
Github user matyix commented on the issue: https://github.com/apache/spark/pull/19775 For those who are still interested using Prometheus you can get the standalone package and source code from here: https://github.com/banzaicloud/spark-metrics . Happy monitoring, try to catch the

[GitHub] spark issue #19775: [SPARK-22343][core] Add support for publishing Spark met...

2017-11-24 Thread matyix
Github user matyix commented on the issue: https://github.com/apache/spark/pull/19775 @jerryshao this PR is not Kubernetes specific, it's an extension of the Spark Metrics system which is part of the core already. We could externalize it the PR #11994 above ever gets merge

[GitHub] spark issue #19775: [SPARK-22343][core] Add support for publishing Spark met...

2017-11-27 Thread matyix
Github user matyix commented on the issue: https://github.com/apache/spark/pull/19775 Hello @felixcheung @jerryshao The PR #11994 generally looks good for adding extensibility into the Metrics system. This PR (Prometheus) works with the changes proposed in PR #11994 out of

[GitHub] spark pull request #19775: Add support for publishing Spark metrics into Pro...

2017-11-17 Thread matyix
GitHub user matyix opened a pull request: https://github.com/apache/spark/pull/19775 Add support for publishing Spark metrics into Prometheus ## What changes were proposed in this pull request? _Originally this PR was submitted to the Spark on K8S fork [here](https