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 7b25b241368 [SPARK-43953][CONNECT] Remove `pass`
7b25b241368 is described below

commit 7b25b241368e3c6cf621785206a479d9f8524234
Author: Bjørn Jørgensen <bjornjorgen...@gmail.com>
AuthorDate: Sat Jun 3 23:18:27 2023 -0700

    [SPARK-43953][CONNECT] Remove `pass`
    
    ### What changes were proposed in this pull request?
    Remove a `pass`
    
    ### Why are the changes needed?
    [Delete this unreachable code or refactor the code to make it 
reachable.](https://sonarcloud.io/project/issues?languages=py&resolved=false&rules=python%3AS1763&id=spark-python&open=AYTDYn77d80yCLHA8rt7)
    and
    [Remove this unneeded 
"pass".](https://sonarcloud.io/project/issues?languages=py&resolved=false&rules=python%3AS2772&id=spark-python&open=AYTDYn77d80yCLHA8rt8)
    
    ### Does this PR introduce _any_ user-facing change?
    No.
    
    ### How was this patch tested?
    Pass GA
    
    Closes #41438 from bjornjorgensen/remove-pass.
    
    Authored-by: Bjørn Jørgensen <bjornjorgen...@gmail.com>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 python/pyspark/sql/connect/plan.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/python/pyspark/sql/connect/plan.py 
b/python/pyspark/sql/connect/plan.py
index 8218faecd9f..2793ecb3272 100644
--- a/python/pyspark/sql/connect/plan.py
+++ b/python/pyspark/sql/connect/plan.py
@@ -1533,7 +1533,6 @@ class WriteOperation(LogicalPlan):
             f"options: '{self.options}'<br />"
             f"</li></ul>"
         )
-        pass
 
 
 class WriteOperationV2(LogicalPlan):


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

Reply via email to