-
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-26055][CORE] InterfaceStability annotations should be retained at runtime #23029
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems OK to me. I don't think there are any downsides to retaining them as they can be freely ignored at runtime by tools that don't care. And these annotations are Java 7+
@@ -17,7 +17,7 @@ | |||
|
|||
package org.apache.spark.annotation; | |||
|
|||
import java.lang.annotation.Documented; | |||
import java.lang.annotation.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No big deal, but I think we'd usually explicitly import 3 classes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually we need to import 5 classes, that's why my IDE turned it into * automatically...
Test build #98810 has finished for PR 23029 at commit
|
retest this please |
Test build #98822 has finished for PR 23029 at commit
|
Since you're here do you want to add |
Test build #98854 has finished for PR 23029 at commit
|
thanks, merging to master! |
… at runtime ## What changes were proposed in this pull request? It's good to have annotations available at runtime, so that tools like MiMa can detect them and deal with then specially. e.g. we don't want to track compatibility for unstable classes. This PR makes `InterfaceStability` annotations to be retained at runtime, to be consistent with `Experimental` and `DeveloperApi` ## How was this patch tested? N/A Closes apache#23029 from cloud-fan/annotation. Authored-by: Wenchen Fan <wenchen@databricks.com> Signed-off-by: Wenchen Fan <wenchen@databricks.com>
What changes were proposed in this pull request?
It's good to have annotations available at runtime, so that tools like MiMa can detect them and deal with then specially. e.g. we don't want to track compatibility for unstable classes.
This PR makes
InterfaceStability
annotations to be retained at runtime, to be consistent withExperimental
andDeveloperApi
How was this patch tested?
N/A