Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15231
I'd prefer `getSQLDataType` over `invokeJava` in the message - `invokeJava`
is likely going to be in every exception and as such would not be useful for
debugging.
Since this is regarding `captureJVMException` which is added here, I think
it would be better to have the message change in this PR. It'd be important for
that information in the message output because otherwise we are degrading the
information we have when an user is reporting an error - it'd be very hard to
troubleshoot `Error in value[[3L]](cond) : Invalid type unknown`. More R side
argument checks as discussed in
https://github.com/apache/spark/pull/15231#discussion_r80594580 would be ok to
be logically in another PR/JIRA.
@HyukjinKwon How about we make the message change here and merge this PR
ASAP? Am I understanding the context or level of the changes required for this?
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
