[GitHub] [spark-docker] Yikun commented on pull request #15: [SPARK-40569] Expose SPARK_MASTER_PORT 7077 for spark standalone cluster

2022-10-27 Thread GitBox
Yikun commented on PR #15: URL: https://github.com/apache/spark-docker/pull/15#issuecomment-1294391023 also cc @holdenk @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

[GitHub] [spark-docker] Yikun commented on pull request #15: [SPARK-40569] Expose SPARK_MASTER_PORT 7077 for spark standalone cluster

2022-10-26 Thread GitBox
Yikun commented on PR #15: URL: https://github.com/apache/spark-docker/pull/15#issuecomment-1292909604 I guess it's ready? Probably the only potential problem is that the 7077 feels a bit hardcode. If other people think there is a problem, we can temporarily not add 7077. Only add

[GitHub] [spark-docker] Yikun commented on pull request #15: [SPARK-40569] Expose SPARK_MASTER_PORT 7077 for spark standalone cluster

2022-10-21 Thread GitBox
Yikun commented on PR #15: URL: https://github.com/apache/spark-docker/pull/15#issuecomment-1286598701 cc @HyukjinKwon @zhengruifeng Mind taking a look, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [spark-docker] Yikun commented on pull request #15: [SPARK-40569] Expose SPARK_MASTER_PORT 7077 for spark standalone cluster

2022-10-20 Thread GitBox
Yikun commented on PR #15: URL: https://github.com/apache/spark-docker/pull/15#issuecomment-1285130451 @dcoliversun Thanks, CI is green. I will take a look soon. A quick question, how many ports should be exposed in future? For common ports, we could write them directly in the