-
Notifications
You must be signed in to change notification settings - Fork 28k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-15290] [BUILD] Move annotations, like @Since / @DeveloperApi, into spark-tags #13074
Conversation
Test build #58464 has finished for PR 13074 at commit
|
Test build #58476 has finished for PR 13074 at commit
|
Test build #58478 has finished for PR 13074 at commit
|
Test build #58480 has finished for PR 13074 at commit
|
cc @JoshRosen |
I'm going to proceed with this since it's better to get it done for 2.0 and enable easier management of annotations going forward. We essentially reviewed this in #12416 and it unblocks SPARK-14613, marked for 2.0 |
Actually josh is on vacation. Definitely go ahead. I looked at it and it seemed ok. I'm not a build expert though. |
Merged to master/2.0 |
…nto spark-tags ## What changes were proposed in this pull request? (See #12416 where most of this was already reviewed and committed; this is just the module structure and move part. This change does not move the annotations into test scope, which was the apparently problem last time.) Rename `spark-test-tags` -> `spark-tags`; move common annotations like `Since` to `spark-tags` ## How was this patch tested? Jenkins tests. Author: Sean Owen <sowen@cloudera.com> Closes #13074 from srowen/SPARK-15290. (cherry picked from commit 122302c) Signed-off-by: Sean Owen <sowen@cloudera.com>
What changes were proposed in this pull request?
(See #12416 where most of this was already reviewed and committed; this is just the module structure and move part. This change does not move the annotations into test scope, which was the apparently problem last time.)
Rename
spark-test-tags
->spark-tags
; move common annotations like@Since
tospark-tags
How was this patch tested?
Jenkins tests.