Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12416#issuecomment-215972386
OK, let's try this PR again but only with the tag changes first. (You're
even welcome to make a separate JIRA that blocks this one, since that much is a
fairly separable change.)
I vote for just leaving these tags in `src/main` then. Although they're
logically test-only, they're just annotations, so can plausibly stay there I
guess. I think we still want to rename the module, ideally. It's just that we
won't make the MLlib changes and won't move the annotations this time, and so
won't need any classifier or type changes in Maven.
Thanks @pravingadakh , my suggestion ended up being the wrong one here.
---
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]
