gaogaotiantian commented on PR #57763: URL: https://github.com/apache/spark/pull/57763#issuecomment-5263308340
I understand that the scripts are for end users, but if we don't believe users should use it through pyspark, we should not ship them. However, if we somehow decide that users that install pyspark should have access to those scripts as well, we can include the full directory. It might be the case. For the current situation, as long as we think those should not be used by end users, we should not ship them. As for `MANIFEST.in`, it has a complicated history with `package_data` I believe. We can take a look at it, but I think that file was shared between all the 3 packages(classic, client, connect) we have? There could be side effects. -- 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]
