Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/5992#issuecomment-117794724
After discussing with @mengxr it sounds like the best thing to do will be
to add an extra argument to checkInputColumn ```msg: String```, with default
value empty string. Then we can pass the extra info as needed. Could you
please modify this PR so it does not move the method, but instead adds the
extra argument? Part of our motivation is that we eventually want these
SchemaUtils to be part of DataFrames, rather than MLlib.
Also, let's not yet modify the uses of checkInputColumn. The reason is
that I want to create abstractions for input/output columns, and then those
abstractions can handle the schema check internally. I'll create JIRAs for
that, to be done after this PR.
Thanks!
---
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]
