You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "LuciferYang (via GitHub)" <gi...@apache.org> on 2023/01/27 05:17:17 UTC
[GitHub] [spark] LuciferYang opened a new pull request, #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
LuciferYang opened a new pull request, #39756:
URL: https://github.com/apache/spark/pull/39756
### What changes were proposed in this pull request?
This pr aims enable GA build and test for `connect-client-jvm` module.
### Why are the changes needed?
`connect-client-jvm` should use GA for testing.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Pass GitHub Actions
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405996923
You can revise and reuse this PR because the last commit is still running.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on a diff in pull request #39756: [DON'T MERGE] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on code in PR #39756:
URL: https://github.com/apache/spark/pull/39756#discussion_r1088201676
##########
.github/workflows/build_and_test.yml:
##########
@@ -152,7 +152,7 @@ jobs:
streaming, sql-kafka-0-10, streaming-kafka-0-10,
mllib-local, mllib,
yarn, mesos, kubernetes, hadoop-cloud, spark-ganglia-lgpl,
- connect, protobuf
+ connect, connect-client-jvm, protobuf
Review Comment:
cc @HyukjinKwon, should we use GA to test `connect-client-jvm` module?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405958908
<img width="615" alt="image" src="https://user-images.githubusercontent.com/1475305/215000924-2fa952e8-fdc1-464d-8eee-d3a76ae295fd.png">
Check manually, `connect-client-jvm` test succeeded
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405993152
> You had better elaborate more narrowly about the PR's contribution.
Got it, it is my bad. I found possible problems after submitting pr, but I didn't update title and pr description in time. I'm very sorry, let me submit another pr for testing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405941790
cc @grundprinzip @hvanhovell @zhenlineo FYI
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1406037319
> I reopened this for you.
Thank you very much. Let me try to reproduce the local test error on the GA in this pr. If it can be reproduced, I will update Pr description and title later
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1406034018
I reopened this for you.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1406074855
I guess you hit the requirement of assembly building. Last time, I asked the similar questions on some `connector` PR. If you hit the local failure again, please try `build/sbt package` first and run the commit.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405970138
[252f93f](https://github.com/apache/spark/pull/39756/commits/252f93f2e6e7f7b977ebfb90265474cf4065c4f4) move `connect-client-jvm` to another group for one more test
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [DON'T MERGE][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405440732
@LuciferYang Please use `Draft` PR instead of `[DON'T MERGE]` tag.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405989352
That sounds totally different from the PR title's claim, `Enable GA build and test for connect-client-jvm module`, @LuciferYang .
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405989594
So I push [252f93f](https://github.com/apache/spark/pull/39756/commits/252f93f2e6e7f7b977ebfb90265474cf4065c4f4) and put them into different groups for testing in order to re-produce the test failure in GA
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1406131084
> I guess you hit the requirement of assembly building. Last time, I asked the similar questions on some `connector` PR. If you hit the local failure again, please try `build/sbt package` first and run the module testing you posted here..
Yes, sbt can solve this problem by this way. Maven needs more steps, such as:
```
build/mvn clean install -pl connector/connect/server -am -DskipTests
build/mvn clean install -pl connector/connect/client/jvm -DskipTests
build/mvn test -pl connector/connect/client/jvm
```
This can solve the problem that the `jar cannot be found`, but there is still `Failed to start the test server on port xxx` problem(sbt local test no this issue). I haven't found the root cause
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405999697
hmm... it seems that a new pr will be submitted. I can't reopen the pr you closed. Let me wait for the test results first
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405981658
Let me close this PR first to prevent accidental merging.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun closed pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun closed pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
URL: https://github.com/apache/spark/pull/39756
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405997547
JIRA issue is the same. I didn't `Close` it so that you can reopen it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405987871
<img width="337" alt="image" src="https://user-images.githubusercontent.com/1475305/215005973-7533ae8e-0da2-4fad-9ebd-bf0baf250497.png">
Thanks @dongjoon-hyun, but will this make `connect` test before `connect-client-jvm`?If so, this may hide some problems. I hope to check `connect-client-jvm` before `connect`, which may cause a test failure. You can run `build/sbt clean "connect-client-jvm/test"` locally to check.
> @LuciferYang I did this already. Please see the master branch result.
>
> * https://github.com/apache/spark/actions/runs/4020447150/jobs/6908393722
>
> ```
> [info] ClientE2ETestSuite:
> [info] - test spark result schema (174 milliseconds)
> ...
> [info] SparkConnectClientSuite:
> [info] - Placeholder test: Create SparkConnectClient (2 milliseconds)
> ...
> [info] DatasetSuite:
> [info] - limit (20 milliseconds)
> ...
> [info] All tests passed.
> ```
I I checked this log and found that `connect` was tested before `connect-client-jvm`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405980963
This is done via SPARK-42046 already, @LuciferYang .
- https://github.com/apache/spark/pull/39549
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1406037942
Got it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [DON'T MERGE][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405443681
Thanks @dongjoon-hyun
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] dongjoon-hyun commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405990707
You had better elaborate more narrowly about the PR's contribution.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Enable GA build and test for `connect-client-jvm` module
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1405998302
Thanks, let me reopen this one and make it as Draft first.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang commented on pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #39756:
URL: https://github.com/apache/spark/pull/39756#issuecomment-1406072383
I can't reproduce the problem I encountered in the local test through GA, because GA has done a lot of preparation work, so that the jar that should not have been found can be found.
Let me close this pr.
Thank you very much for your help @dongjoon-hyun
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] LuciferYang closed pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang closed pull request #39756: [SPARK-42200][INFRA] Put `connect-client-jvm` into separate test groups and xxx
URL: https://github.com/apache/spark/pull/39756
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org