Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/4632
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-168112595
I'm going to close this pull request. If this is still relevant and you are
interested in pushing it forward, please open a new pull request. Thanks!
---
If your project
Github user iteratec-hh commented on a diff in the pull request:
https://github.com/apache/spark/pull/4632#discussion_r44517585
--- Diff: core/src/main/scala/org/apache/spark/metrics/MetricsConfig.scala
---
@@ -57,14 +59,16 @@ private[spark] class MetricsConfig(conf: SparkConf)
ex
Github user iteratec-hh commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-155481980
I'm also very interested in this config-param! I'm looking forward to get
this in next release :)
+1
---
If your project is set up for it, you can reply to this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154289989
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154289985
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154289768
**[Test build #45193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45193/consoleFull)**
for PR 4632 at commit
[`7d0204e`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154273416
**[Test build #45193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45193/consoleFull)**
for PR 4632 at commit
[`7d0204e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154272305
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154272326
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154272233
Does `docs/monitoring.md` need an update to include the new config?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-154272194
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/4632#discussion_r44099630
--- Diff: core/src/main/scala/org/apache/spark/metrics/MetricsConfig.scala
---
@@ -57,14 +59,16 @@ private[spark] class MetricsConfig(conf: SparkConf)
extends
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/4632#discussion_r44099446
--- Diff: core/src/main/scala/org/apache/spark/metrics/MetricsConfig.scala
---
@@ -57,14 +59,16 @@ private[spark] class MetricsConfig(conf: SparkConf)
extends
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/4632#discussion_r44099482
--- Diff: core/src/main/scala/org/apache/spark/metrics/MetricsSystem.scala
---
@@ -124,7 +124,7 @@ private[spark] class MetricsSystem private (
*
Github user paulbes commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-153745576
Being able to configure a fixed set of metrics names is something we are
also interested in, what is the progress on this work?
---
If your project is set up for it, yo
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140236069
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42444/testReport/
---
If your project is set up for it, you can reply to this email and have your
repl
Github user ryan-williams commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140235859
I'm not sure how to figure out what test failed from the Jenkins output
here
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42444/consoleFull
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140233884
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140233883
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140233827
[Test build #42444 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42444/console)
for PR 4632 at commit
[`7d0204e`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140209080
[Test build #42444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42444/consoleFull)
for PR 4632 at commit
[`7d0204e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140208347
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140208319
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140182809
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140182808
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140182767
[Test build #42437 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42437/console)
for PR 4632 at commit
[`578162b`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140180688
[Test build #42437 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42437/consoleFull)
for PR 4632 at commit
[`578162b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140179925
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-140179948
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-136974419
I have little knowledge of the metrics system. I can appreciate the
overhead of yet another config flag, and can also see how you might want to
monitor apps by instance (
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-136915532
Perhaps @vanzin or @srowen can look at this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your p
Github user dajac commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-101239287
Hey @pwendell and @JoshRosen, what is the status on this?
I would like to monitor all my recurrent jobs and having fixed set of
metrics which doesn't change betwee
Github user ryan-williams commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-85180665
Thanks @pwendell. I had stumbled across that
[SPARK-3377](https://issues.apache.org/jira/browse/SPARK-3377) work as well.
I think there are solid arguments
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-84209577
Hey @JoshRosen can probably take a look at this. One thing though, IIRC the
reason why we have a unique ID here is because some sets of users requested the
exact opposit
Github user ryan-williams commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-82420712
bump, @pwendell any ideas who would be a good reviewer for this?
This is a pretty important piece of my
[`grafana-spark-dashboards`](https://github.com/hamm
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-76455746
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-76455726
[Test build #28076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28076/consoleFull)
for PR 4632 at commit
[`64a7eee`](https://gith
Github user ryan-williams commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-76444840
This one is ready for review, I think
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-76441577
[Test build #28076 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28076/consoleFull)
for PR 4632 at commit
[`64a7eee`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74588350
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74588341
[Test build #27581 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27581/consoleFull)
for PR 4632 at commit
[`00df060`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74578739
[Test build #27581 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27581/consoleFull)
for PR 4632 at commit
[`00df060`](https://githu
Github user ryan-williams commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74578411
scala style check should pass now
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project do
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74577799
[Test build #27578 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27578/consoleFull)
for PR 4632 at commit
[`ffdc06a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74577802
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/4632#issuecomment-74577467
[Test build #27578 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27578/consoleFull)
for PR 4632 at commit
[`ffdc06a`](https://githu
GitHub user ryan-williams opened a pull request:
https://github.com/apache/spark/pull/4632
[SPARK-5847] Allow for namespacing metrics by conf params other than
spark.app.id
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ryan-wi
48 matches
Mail list logo