Github user steveloughran closed the pull request at:
https://github.com/apache/spark/pull/17747
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127024047
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -728,6 +768,103 @@ private[history] class FsHistoryProvider(conf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127020940
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -164,6 +169,16 @@ private[history] class FsHistoryProvider(conf:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127019974
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/ApplicationHistoryProvider.scala
---
@@ -100,6 +102,13 @@ private[history] abstract class
App
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127024292
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -728,6 +768,103 @@ private[history] class FsHistoryProvider(conf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127020761
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -129,6 +131,9 @@ private[history] class FsHistoryProvider(conf:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127023518
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -728,6 +768,103 @@ private[history] class FsHistoryProvider(conf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127022304
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -240,43 +253,52 @@ private[history] class FsHistoryProvider(conf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127023652
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -728,6 +768,103 @@ private[history] class FsHistoryProvider(conf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r127023411
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -728,6 +768,103 @@ private[history] class FsHistoryProvider(conf
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r122295075
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -164,6 +169,16 @@ private[history] class FsHistoryProvide
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r122294995
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -129,6 +131,9 @@ private[history] class FsHistoryProvider
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r122279400
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -110,6 +117,14 @@ class HistoryServer(
appCache.getS
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r122256050
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryMetricSource.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Licensed to the Apache
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r120972491
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -164,6 +169,16 @@ private[history] class FsHistoryProvider(conf:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r120969616
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryMetricSource.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Licensed to the Apache Softwar
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r120971242
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryServer.scala ---
@@ -110,6 +117,14 @@ class HistoryServer(
appCache.getSparkUI(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r120972081
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -129,6 +131,9 @@ private[history] class FsHistoryProvider(conf:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17747#discussion_r120969440
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/HistoryMetricSource.scala
---
@@ -0,0 +1,166 @@
+/*
+ * Licensed to the Apache Softwar
GitHub user steveloughran opened a pull request:
https://github.com/apache/spark/pull/17747
[SPARK-11373] [CORE] Add metrics to the FsHistoryProvider
## What changes were proposed in this pull request?
This adds metrics to the history server, with the `FsHistoryProvider`
me
20 matches
Mail list logo