-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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-13152][CORE] Fix task metrics deprecation warning #11056
[SPARK-13152][CORE] Fix task metrics deprecation warning #11056
Conversation
…sRead so we don't have unecessary deprecation warnings in our build.
also cc @srowen who was interested |
@@ -81,10 +81,14 @@ class InputMetrics private ( | |||
*/ | |||
def readMethod: DataReadMethod.Value = DataReadMethod.withName(_readMethod.localValue) | |||
|
|||
// Once incBytesRead & intRecordsRead is ready to be removed from the public API | |||
// we can remove the internal versions and make the previous public API private. |
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.
can you add that this is to suppress warnings?
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.
Done :)
LGTM |
Test build #50677 has finished for PR 11056 at commit
|
retest this please |
Test build #50683 has finished for PR 11056 at commit
|
Test build #50692 has finished for PR 11056 at commit
|
Merged into master |
Make an internal non-deprecated version of incBytesRead and incRecordsRead so we don't have unecessary deprecation warnings in our build.
Right now incBytesRead and incRecordsRead are marked as deprecated and for internal use only. We should make private[spark] versions which are not deprecated and switch to those internally so as to not clutter up the warning messages when building.
cc @andrewor14 who did the initial deprecation