Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
Let's continue at https://github.com/apache/spark/pull/21995
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
I see that `zookeeper.connect` is set to `[zkhost]:[zkport]` after ZK is
started, so looks like it's trying to do so. That much hasn't changed at least.
I can increase the timeout; although I don't
Github user ijuma commented on the issue:
https://github.com/apache/spark/pull/21955
Looks similar to https://issues.apache.org/jira/browse/KAFKA-7193, are we
using the same ZK ip in the server and client?
---
-
To
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/21955
I don't see an obvious issue. Looks like zookeeper.connection.timeout.ms
isn't being set, so it's defaulting to 6 seconds... could try tweaking it
upwards.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
Hm, I might need your eyes on this again @koeninger @zsxwing or even @ijuma
. I see this failure in the test, which looks like it's no longer happy
starting ZK in some cases. Does that ring any
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
I believe the second-to-last 'pass' is the authoritative one. The only
change difference between the two runs is an added comment in an XML file. I'm
going to proceed as this might fix up a test,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94087/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94086/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
**[Test build #94087 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94087/testReport)**
for PR 21955 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94086 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94086/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94072/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
**[Test build #94072 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94072/testReport)**
for PR 21955 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94087 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94087/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
**[Test build #94086 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94086/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user koeninger commented on the issue:
https://github.com/apache/spark/pull/21955
jenkins, retest this please
---
-
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/21955
**[Test build #94051 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94051/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94051/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
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/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94072 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94072/testReport)**
for PR 21955 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/21955
There are some configs need to tune to make the tests faster. See:
https://github.com/apache/spark/pull/21488/files#diff-1aecc5a638048c27f97cbd9579cb53b5R311
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94051 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94051/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94048/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
**[Test build #94048 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94048/testReport)**
for PR 21955 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94048 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94048/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ijuma commented on the issue:
https://github.com/apache/spark/pull/21955
@srowen I think you're pretty close, I added a few comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
**[Test build #94031 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94031/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94031/
Test FAILed.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
BTW this only passes, I think, because I snipped out part of the log
compaction test. I don't think it's valid to merge as is. Pinging @koeninger in
case he's willing and able to weigh in on how to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94031 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94031/testReport)**
for PR 21955 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #94012 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94012/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94012/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
**[Test build #94012 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94012/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
@ijuma heh yeah I just pushed this to a branch to see what would happen
before signing off. Not quite there. I pushed fixes for a few of the obvious
compile issues, but might need a Kafka expert to
Github user ijuma commented on the issue:
https://github.com/apache/spark/pull/21955
Looks like some test code is using internal Kafka classes that have changed
or have been removed:
> [error]
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93935/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #93935 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93935/testReport)**
for PR 21955 at commit
Github user tedyu commented on the issue:
https://github.com/apache/spark/pull/21955
Thanks for the follow-up.
lgtm
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21955
**[Test build #93935 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93935/testReport)**
for PR 21955 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21955
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/21955
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21955
@tedyu @zsxwing check my work here. The other module needed to update too
right?
---
-
To unsubscribe, e-mail:
Github user holdensmagicalunicorn commented on the issue:
https://github.com/apache/spark/pull/21955
@srowen, thanks! I am a bot who has found some folks who might be able to
help with the review:@tdas, @timvw and @vanzin
---
63 matches
Mail list logo