Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15677
Sorry, I am confusing the merging standards. Why can we accept merging
something incorrect?
Just like `reverse`, to the external users, the argument type of `str` is
not `a string expression`. How can our users know some types are implicitly
casted but some other types are not able to cast? This is not the only example.
We have many cases like this.
Anyway, if this PR is merged, I hope we keep the same standard in the other
PRs. Based on my observations, we reversed multiple PRs that improved the codes
but did not perfectly resolved the problem.
---
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
