itholic commented on code in PR #40642:
URL: https://github.com/apache/spark/pull/40642#discussion_r1329417329


##########
python/pyspark/errors/error_classes.py:
##########
@@ -24,6 +24,21 @@
       "Argument `<arg_name>` is required when <condition>."
     ]
   },
+  "CANNOT_ACCESS_TO_DUNDER": {

Review Comment:
   We can update it as a follow-up if there is any better suggestions for the 
name of error classes (It's not a API changes but only affect to error message, 
so it would be fine)
   
   Which one do you prefer `CANNOT_ACCESS_DUNDER` or 
`ILLEGAL_ACCESS_TO_DUNDER`? Can you pick one of them, or even better idea? Then 
I can make a follow-up for you.



-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to