Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98909/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #98909 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98909/testReport)**
for PR 22138 at commit
[`fd4ff83`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #98909 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98909/testReport)**
for PR 22138 at commit
[`fd4ff83`](https://github.com/apache/spark/commit/fd
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #98904 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98904/testReport)**
for PR 22138 at commit
[`fd4ff83`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98904/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #98904 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98904/testReport)**
for PR 22138 at commit
[`fd4ff83`](https://github.com/apache/spark/commit/fd
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
@zsxwing
Given that Spark 2.4 vote passes, could we revisit and make progress on
this?
---
-
To unsubscribe, e-mail: re
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
@gaborgsomogyi Yeah... I'm just waiting for it. Btw I proposed solution on
SPARK-10816 as well and it is also waiting for response. I'm going to work on
another item or review others so that I c
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/22138
> Do you have more to review?
No. BTW I've just seen a -1 for 2.4 RC...
---
-
To unsubscribe, e-mail: reviews-uns
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Kindly ask for reviewing. Please never mind when you're busy with fixing
bugs on Spark 2.4 RC.
@gaborgsomogyi I guess I left two things for committer decision: 1. define
soft boundary
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96695/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96695 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96695/testReport)**
for PR 22138 at commit
[`d3f097b`](https://github.com/apache/spark/commit/d
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Just rebased.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96695 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96695/testReport)**
for PR 22138 at commit
[`d3f097b`](https://github.com/apache/spark/commit/d3
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
@gaborgsomogyi Totally makes sense. Let me address while the patch is
reviewed by committers. I may get recommendations to rename the config or even
more, so addressing documentation would be th
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/22138
One more thing just came to my mind is the documentation. The parameter
documentation is a gap even for the original feature. As it has been grown and
several additional parameters added it wo
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96337/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96337/testReport)**
for PR 22138 at commit
[`ddd4f2f`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96337 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96337/testReport)**
for PR 22138 at commit
[`ddd4f2f`](https://github.com/apache/spark/commit/dd
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96317/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96317 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96317/testReport)**
for PR 22138 at commit
[`ddd4f2f`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96317 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96317/testReport)**
for PR 22138 at commit
[`ddd4f2f`](https://github.com/apache/spark/commit/dd
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96300/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96300/testReport)**
for PR 22138 at commit
[`ddd4f2f`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96300/testReport)**
for PR 22138 at commit
[`ddd4f2f`](https://github.com/apache/spark/commit/dd
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96295/testReport)**
for PR 22138 at commit
[`da5bd7f`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96295/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96295 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96295/testReport)**
for PR 22138 at commit
[`da5bd7f`](https://github.com/apache/spark/commit/da
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96250/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96250 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96250/testReport)**
for PR 22138 at commit
[`d2f07b7`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96250 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96250/testReport)**
for PR 22138 at commit
[`d2f07b7`](https://github.com/apache/spark/commit/d2
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
> just wondering why org.apache.spark.sql.kafka010.CachedKafkaProducer uses
com.google.common.cache.LoadingCache?
Because KafkaProducer is thread-safe unless it enables transaction, henc
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/22138
No objection about `org.apache.commons.pool2.impl.GenericKeyedObjectPool`
stuff but just wondering why
`org.apache.spark.sql.kafka010.CachedKafkaProducer` uses
`com.google.common.cache.Loadin
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Now vote for Spark 2.4 is in progress. If we are not in stand-by mode for
any blocker issues for Spark 2.4 RC, I'd be really happy if someone could
revisit this and continue reviewing.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95800/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95800 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95800/testReport)**
for PR 22138 at commit
[`107880e`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95800 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95800/testReport)**
for PR 22138 at commit
[`107880e`](https://github.com/apache/spark/commit/10
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95784/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95784 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95784/testReport)**
for PR 22138 at commit
[`2da9924`](https://github.com/apache/spark/commit/2
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Regarding metrics in FetchedDataPool, I just add basic metrics so that
tests can leverage on verification. I was adding numActive as well as numIdle,
but tracking and measuring them needs more r
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95784 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95784/testReport)**
for PR 22138 at commit
[`2da9924`](https://github.com/apache/spark/commit/2d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95777/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95777 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95777/testReport)**
for PR 22138 at commit
[`5b56bd0`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95777 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95777/testReport)**
for PR 22138 at commit
[`5b56bd0`](https://github.com/apache/spark/commit/5b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95744/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95744 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95744/testReport)**
for PR 22138 at commit
[`b47e99b`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95744 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95744/testReport)**
for PR 22138 at commit
[`b47e99b`](https://github.com/apache/spark/commit/b4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95688/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95688 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95688/testReport)**
for PR 22138 at commit
[`9685cc5`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95688 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95688/testReport)**
for PR 22138 at commit
[`9685cc5`](https://github.com/apache/spark/commit/96
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
@zsxwing
If it means code freeze for 2.4 is just around the corner then sure! We can
focus on blockers for releasing 2.4, and revisit this again. Let me reflect
@gaborgsomogyi review commen
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22138
Thanks for your PR. This is really a big change. It will need very careful
review as it changes a lot of critical code path and the current Kafka consumer
logic is really complicated. Let's hold thi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95525/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95525 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95525/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95525 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95525/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/01
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Retest this, please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22138
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95513/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95513 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95513/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95513 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95513/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/01
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Two tests failed and not relevant to this patch.
* org.apache.spark.scheduler.DAGSchedulerSuite.Barrier task failures from
the same stage attempt don't trigger multiple stage retries
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95502/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95502 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95502/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95502 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95502/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/01
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
From the stack trace of test failure, it doesn't look like relevant to the
code change I guess. Jenkins shows REGRESSION but the test is added at Aug 25,
2018 so aged less than 7 days, which is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95474/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95474/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/0
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Updated the description of PR to reflect the new approach.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95474 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95474/testReport)**
for PR 22138 at commit
[`017c0bb`](https://github.com/apache/spark/commit/01
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
I just applied a new approach: "separate of concerns". This approach does
pooling for both kafka consumers as well as fetched data.
Both pools support eviction on idle objects, which wil
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95472 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95472/testReport)**
for PR 22138 at commit
[`648550a`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95472/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95472 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95472/testReport)**
for PR 22138 at commit
[`648550a`](https://github.com/apache/spark/commit/64
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95450/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95450 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95450/testReport)**
for PR 22138 at commit
[`7d13ee5`](https://github.com/apache/spark/commit/7
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Thanks @attilapiros I fixed the missing spots you left comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apac
1 - 100 of 134 matches
Mail list logo