Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
@srowen Thank you very much.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22670
Merged to master
---
-
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/22670
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97473/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22670
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/22670
**[Test build #97473 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97473/testReport)**
for PR 22670 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22670
**[Test build #97473 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97473/testReport)**
for PR 22670 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22670
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/22670
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
@srowen OK, Let me look.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22670
I don't so much mean that much refactoring. I wonder if there are 1-2 other
places where common Kafka params are set in tests that we could add this to for
now, that kind of thing. This change is OK
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
@srowen Thanks. Did you mean, the test cases should extend a shared spark
context (SharedKafkaSparkContext) which would have this property set ?
Actually Sean, there are 3 suites in
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22670
@dilipbiswal I like this change too. The suite goes from 4:34 to 0:53. I
wonder if we can make this change elsewhere in general Kafka test config? This
kind of setting seems useful everywhere.
---
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
cc @gatorsmile
---
-
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/22670
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/22670
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97102/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22670
**[Test build #97102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97102/testReport)**
for PR 22670 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22670
**[Test build #97102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97102/testReport)**
for PR 22670 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22670
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/22670
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
19 matches
Mail list logo