raydp icon indicating copy to clipboard operation
raydp copied to clipboard

Two users cannot run their Spark driver on the same node and connect to the same Ray cluster.

Open Bowen0729 opened this issue 2 years ago • 7 comments

https://github.com/oap-project/raydp/issues/175#issuecomment-1195190398

two users cannot run their Spark driver on the same node and connect to the same Ray cluster.

Start the java gateway with a random port.

Bowen0729 avatar Jul 31 '22 14:07 Bowen0729

Thank you @Bowen0729 , can you please add a test for this?

carsonwang avatar Aug 01 '22 01:08 carsonwang

Thank you @Bowen0729 , can you please add a test for this?

Hi, I have added a test for this.

Bowen0729 avatar Aug 01 '22 08:08 Bowen0729

The CI is failed. What's the problem?

Bowen0729 avatar Aug 02 '22 02:08 Bowen0729

The CI is flaky these days. I just triggered a rerun of the failed test.

carsonwang avatar Aug 02 '22 02:08 carsonwang

@carsonwang My test is passed, other tests on macos error.

Bowen0729 avatar Aug 11 '22 11:08 Bowen0729

@Bowen0729 Previously I saw you tried to test running two spark drivers on the same node, that is the test we want to have. Is there any challenge to do that?

carsonwang avatar Aug 12 '22 02:08 carsonwang

@Bowen0729 Previously I saw you tried to test running two spark drivers on the same node, that is the test we want to have. Is there any challenge to do that?

Yes, the test can pass on my mac, but cannot pass for the CI, I think I need some help.

Bowen0729 avatar Aug 12 '22 06:08 Bowen0729

Sorry, I've been a little busy recently, so I don't have time to solve this problem. Thanks for helping.

Bowen0729 avatar Sep 02 '22 05:09 Bowen0729

@carsonwang the test still hangs sometime. Shall we skip the tests?

kira-lin avatar Sep 07 '22 08:09 kira-lin

Ok. Let's skip it for now. Does it also hangs sometime on your local machine?

carsonwang avatar Sep 07 '22 09:09 carsonwang

yes, it hangs sometime and when I ctrl-c it, it shows popen_fork, even if I should have set start_method to spawn.

kira-lin avatar Sep 07 '22 09:09 kira-lin

Thanks all! Merged

carsonwang avatar Sep 08 '22 10:09 carsonwang