Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20407#discussion_r167392027
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -156,6 +156,15 @@ object SQLConf {
.booleanConf
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20519
I am not yet fully sure about how sitecustomize fills some bytes to stdout
but I believe it doesn't always happen because at least we have it for Python
coverage -
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20531
**[Test build #87280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87280/testReport)**
for PR 20531 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20531
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20531
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/766/
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20531#discussion_r167391819
--- Diff: python/pyspark/sql/tests.py ---
@@ -4509,23 +4523,32 @@ def weighted_mean(v, w):
return weighted_mean
def
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20561
**[Test build #87279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87279/testReport)**
for PR 20561 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20561
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/765/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20561
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20561
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20555#discussion_r167391619
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -258,54 +262,43 @@ public int read(byte[] b, int offset, int len) throws
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20559#discussion_r167391496
--- Diff: python/pyspark/sql/tests.py ---
@@ -2867,6 +2867,35 @@ def
test_create_dataframe_required_pandas_not_found(self):
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20559#discussion_r167391409
--- Diff: python/pyspark/sql/types.py ---
@@ -1766,15 +1781,13 @@ def _check_series_convert_timestamps_localize(s,
from_timezone, to_timezone):
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20555#discussion_r167391309
--- Diff: core/src/main/java/org/apache/spark/io/ReadAheadInputStream.java
---
@@ -232,7 +229,9 @@ private void waitForAsyncReadComplete() throws
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87278/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20559
**[Test build #87278 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87278/testReport)**
for PR 20559 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20559
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20559
**[Test build #87278 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87278/testReport)**
for PR 20559 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20559
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/764/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20559
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20559
@BryanCutler Seems like pandas handles `tzlocal()` differently than other
timezone, and it might handle DST incorrectly, I guess.
---
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20559#discussion_r167390815
--- Diff: python/pyspark/sql/types.py ---
@@ -1709,6 +1709,15 @@ def _check_dataframe_convert_date(pdf, schema):
return pdf
+def
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20561
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20561
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87277/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20561
**[Test build #87277 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87277/testReport)**
for PR 20561 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20559#discussion_r167389708
--- Diff: python/pyspark/sql/types.py ---
@@ -1709,6 +1709,15 @@ def _check_dataframe_convert_date(pdf, schema):
return pdf
+def
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20559#discussion_r167389710
--- Diff: python/pyspark/sql/tests.py ---
@@ -4124,7 +4126,7 @@ def test_vectorized_udf_timestamps(self):
data = [(0, datetime(1969, 1, 1, 1, 1,
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20559#discussion_r167389709
--- Diff: python/pyspark/sql/types.py ---
@@ -1709,6 +1709,15 @@ def _check_dataframe_convert_date(pdf, schema):
return pdf
+def
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19788
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dongjoon-hyun closed the pull request at:
https://github.com/apache/spark/pull/20563
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20563
Thank you so much!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20565
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20565
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87276/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20565
**[Test build #87276 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87276/testReport)**
for PR 20565 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20561
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/763/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20561
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20561
**[Test build #87277 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87277/testReport)**
for PR 20561 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20561#discussion_r167387192
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/UnsafeKVExternalSorter.java
---
@@ -98,10 +99,20 @@ public UnsafeKVExternalSorter(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20561#discussion_r167387138
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/UnsafeKVExternalSorterSuite.scala
---
@@ -205,4 +206,42 @@ class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20561#discussion_r167387073
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/UnsafeKVExternalSorter.java
---
@@ -98,10 +99,20 @@ public UnsafeKVExternalSorter(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20490
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20490#discussion_r167386197
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2.scala
---
@@ -116,21 +118,45 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20490#discussion_r167386169
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2.scala
---
@@ -116,21 +118,45 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20490#discussion_r167386125
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/DataSourceWriter.java
---
@@ -78,10 +88,11 @@ default void
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20490#discussion_r167386061
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/DataSourceWriter.java
---
@@ -78,10 +88,11 @@ default void
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20563
thanks, merging to 2.2!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167385858
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -37,22 +100,129 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20564
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87275/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20564
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167385829
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -37,22 +100,129 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20564
**[Test build #87275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87275/testReport)**
for PR 20564 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167385722
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,80 @@
package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87274/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20387
**[Test build #87274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87274/testReport)**
for PR 20387 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20565
**[Test build #87276 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87276/testReport)**
for PR 20565 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20565
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/762/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20565
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user liufengdb opened a pull request:
https://github.com/apache/spark/pull/20565
SPAR[SPARK-23379][SQL] remove redundant metastore access
## What changes were proposed in this pull request?
If the target database name is as same as the current database, we should
be
Github user liufengdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/20564#discussion_r167381054
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1107,11 +1107,6 @@ private[spark] class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20564#discussion_r167380287
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1107,11 +1107,6 @@ private[spark] class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20407#discussion_r167378911
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -156,6 +156,15 @@ object SQLConf {
.booleanConf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20564
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/761/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20564
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20562
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20564
**[Test build #87275 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87275/testReport)**
for PR 20564 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20562
Thanks! Merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
GitHub user liufengdb opened a pull request:
https://github.com/apache/spark/pull/20564
[SPARK-23378][SQL] move setCurrentDatabase from HiveExternalCatalog to
HiveClientImpl
## What changes were proposed in this pull request?
This enforces the rule that no calls from
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20550
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20550
Merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20550
**[Test build #4090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4090/testReport)**
for PR 20550 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20563
cc @cloud-fan and @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20387
**[Test build #87274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87274/testReport)**
for PR 20387 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/760/
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167374952
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,131 @@
package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20563
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167374648
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,131 @@
package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20563
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87273/
Test PASSed.
---
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167374579
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,131 @@
package
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20563
**[Test build #87273 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87273/testReport)**
for PR 20563 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20562
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87272/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20562
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20562
**[Test build #87272 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87272/testReport)**
for PR 20562 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r167369570
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,131 @@
package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20490
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87269/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20490
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20490
**[Test build #87269 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87269/testReport)**
for PR 20490 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20351
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87271/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20351
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20351
**[Test build #87271 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87271/testReport)**
for PR 20351 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20519
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20519
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87267/
Test PASSed.
---
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359005
--- Diff:
core/src/main/scala/org/apache/spark/metrics/sink/PrometheusSink.scala ---
@@ -0,0 +1,160 @@
+/*
+ * Licensed to the Apache Software
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359036
--- Diff:
core/src/main/java/org/apache/spark/metrics/prometheus/client/exporter/TextFormatWithTimestamp.java
---
@@ -0,0 +1,178 @@
+/*
+ *
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359067
--- Diff:
core/src/main/java/org/apache/spark/metrics/prometheus/client/exporter/PushGatewayWithTimestamp.java
---
@@ -0,0 +1,320 @@
+/*
+ *
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359075
--- Diff:
core/src/main/java/org/apache/spark/metrics/prometheus/client/exporter/PushGatewayWithTimestamp.java
---
@@ -0,0 +1,320 @@
+/*
+ *
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359048
--- Diff:
core/src/main/java/org/apache/spark/metrics/prometheus/client/exporter/TextFormatWithTimestamp.java
---
@@ -0,0 +1,178 @@
+/*
+ *
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359010
--- Diff:
core/src/main/scala/org/apache/spark/metrics/sink/PrometheusSink.scala ---
@@ -0,0 +1,160 @@
+/*
+ * Licensed to the Apache Software
Github user stoader commented on a diff in the pull request:
https://github.com/apache/spark/pull/19775#discussion_r167359085
--- Diff:
core/src/main/java/org/apache/spark/metrics/prometheus/client/exporter/PushGatewayWithTimestamp.java
---
@@ -0,0 +1,320 @@
+/*
+ *
1 - 100 of 325 matches
Mail list logo