This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 8d70f4ba5396 [SPARK-48107][PYTHON] Exclude tests from Python 
distribution
8d70f4ba5396 is described below

commit 8d70f4ba53962de540fb3dc5bdedd32754be974d
Author: Nicholas Chammas <nicholas.cham...@gmail.com>
AuthorDate: Thu May 2 23:52:43 2024 -0700

    [SPARK-48107][PYTHON] Exclude tests from Python distribution
    
    ### What changes were proposed in this pull request?
    
    Change the Python manifest so that tests are excluded from the packages 
that are built for distribution.
    
    ### Why are the changes needed?
    
    Tests were unintentionally included in the distributions as part of #44920. 
See [this 
comment](https://github.com/apache/spark/pull/44920/files#r1586979834).
    
    ### Does this PR introduce _any_ user-facing change?
    
    No, since #44920 hasn't been released to any users yet.
    
    ### How was this patch tested?
    
    I built Python packages and inspected `SOURCES.txt` to confirm that tests 
were excluded:
    
    ```sh
    cd python
    rm -rf pyspark.egg-info || echo "No existing egg info file, skipping 
deletion"
    python3 packaging/classic/setup.py sdist
    python3 packaging/connect/setup.py sdist
    find dist -name '*.tar.gz' | xargs -I _ tar xf _ --directory=dist
    cd ..
    open python/dist
    find python/dist -name SOURCES.txt | xargs code
    ```
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #46354 from nchammas/SPARK-48107-package-json.
    
    Authored-by: Nicholas Chammas <nicholas.cham...@gmail.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 python/MANIFEST.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/python/MANIFEST.in b/python/MANIFEST.in
index 0374b3096d47..45c9dca8b474 100644
--- a/python/MANIFEST.in
+++ b/python/MANIFEST.in
@@ -16,7 +16,7 @@
 
 # Reference: https://setuptools.pypa.io/en/latest/userguide/miscellaneous.html
 
-graft pyspark
+recursive-include pyspark *.pyi py.typed *.json
 recursive-include deps/jars *.jar
 graft deps/bin
 recursive-include deps/sbin spark-config.sh spark-daemon.sh 
start-history-server.sh stop-history-server.sh


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to