[GitHub] [spark] tbcs opened a new pull request #24448: [SPARK-23299][SQL][PYSPARK] Fix __repr__ behaviour for Rows

[GitBox <gi...@apache.org>]

tbcs opened a new pull request #24448: [SPARK-23299][SQL][PYSPARK] Fix __repr__ behaviour for Rows
URL: https://github.com/apache/spark/pull/24448
 
 
   This is PR is meant to replace #20503, which lay dormant for a while.  The solution in the original PR is still valid, so this is just that patch rebased onto the current master.
   
   Original summary follows.
   
   ## What changes were proposed in this pull request?
   
   Fix `__repr__` behaviour for Rows.
   
   Rows `__repr__` assumes data is a string when column name is missing.
   Examples,
   
   ```
   >>> from pyspark.sql.types import Row
   >>> Row ("Alice", "11")
   <Row(Alice, 11)>
   
   >>> Row (name="Alice", age=11)
   Row(age=11, name='Alice')
   
   >>> Row ("Alice", 11)
   <snip stack trace>
   TypeError: sequence item 1: expected string, int found
   ```
   
   This is because Row () when called without column names assumes everything is a string.
   
   ## How was this patch tested?
   
   Manually tested and a unit test was added to `python/pyspark/sql/tests/test_types.py`.
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org