Github user srowen commented on the issue:

    https://github.com/apache/spark/pull/15677
  
    @gatorsmile Taking your example, before, the user gets no information at 
all about the type, except that it looks maybe string-like. Now at least we say 
it's a 'string expression' which suggests, correctly, that you can't send it a 
number. I understand that 'string expression' isn't tightly defined. But it 
wasn't defined at all before; it's 'incorrect' now, if you like, and less so 
after.
    
    Are you essentially saying remove all of the "Arguments" sections? If that 
would unblock the discussion, OK, that's an option. @HyukjinKwon how do you 
feel about that. @gatorsmile would you be willing to follow this up then with a 
PR to restore Arguments sections with whatever you believe should be said about 
them?
    
    It's a big change, it's an important change, I am looking forward to 
getting these updates in place.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to