You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thanks @slfan1989 for this effort, it means a lot to the Tez project!
I believe "Tez" related stuff cannot stay in the hadoop repository, so if you think the unit test provided makes sense, you can adapt it to a generic one
also double-checked that adding TimelineEntityWriter by config.register makes the unit test work, however, to stay consistent, implementing the TimelineEntityReader also makes sense
thanks @slfan1989 for this effort, it means a lot to the Tez project! I believe "Tez" related stuff cannot stay in the hadoop repository, so if you think the unit test provided makes sense, you can adapt it to a generic one
also double-checked that adding TimelineEntityWriter by config.register makes the unit test work, however, to stay consistent, implementing the TimelineEntityReader also makes sense
@abstractdog Thank you for the information! I will record you as a co-author of this PR. My understanding of the timeline might not be fully comprehensive. During the upgrade of Jersey to version 2.x, my primary focus was on adapting the unit tests bundled with the Hadoop project. Please feel free to share any additional feedback or details. I will continue to follow up on issues related to Jersey 2.x.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
JIRA: YARN-11877. Resolve JAXB IllegalAnnotation issue in TimelineEntity with Jersey 2.x.
How was this patch tested?
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?