Github user MrBago commented on the issue:
https://github.com/apache/spark/pull/20280
I think we should raise an error if `__from_dict__` is set and the user
tries to index using a position or a slice.
Indexing by field name takes the same code path for Rows that are and are
not `__from_dict__` and I don't think we can "discourage" this because
`row.fieldName` is commonly used in our docs and other pyspark learning
materials. If performance is an issue, maybe we should replace `__fields__`
with a dict that maps fileldName -> position.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
