nchammas commented on code in PR #57763:
URL: https://github.com/apache/spark/pull/57763#discussion_r3720802254
##########
python/packaging/classic/setup.py:
##########
@@ -342,14 +342,7 @@ def run(self):
package_data={
"pyspark.jars": ["*.jar"],
"pyspark.bin": ["*"],
- "pyspark.sbin": [
- "spark-config.sh",
- "spark-daemon.sh",
- "start-connect-server.sh",
- "start-history-server.sh",
- "stop-connect-server.sh",
- "stop-history-server.sh",
- ],
+ "pyspark.sbin": ["*"],
Review Comment:
Correct. Per the PR description and my comment just above, I am proposing we
make it a clear project policy not to support launching clusters with PySpark
vs. micro-managing what gets packaged. The latter is more annoying to maintain
and leads to the packaging bug described in the PR description.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
