Skip to content

Conversation

@dineshchitlangia
Copy link
Contributor

No description provided.

@dineshchitlangia
Copy link
Contributor Author

/label ozone

@elek elek added the ozone label Aug 28, 2019
Enforce secret length using Preconditions.
@dineshchitlangia
Copy link
Contributor Author

/retest

@dineshchitlangia
Copy link
Contributor Author

Addressed review comment from @anuengineer to avoid padding and instead enforce the secret length.

@dineshchitlangia
Copy link
Contributor Author

Failures unrelated to patch

Copy link
Contributor

@bharatviswa504 bharatviswa504 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posted a few comments in place.

@bharatviswa504
Copy link
Contributor

+1 LGTM. Pending CI.

@dineshchitlangia
Copy link
Contributor Author

failures unrelated to patch

@bharatviswa504 bharatviswa504 merged commit 46696bd into apache:trunk Aug 30, 2019
@bharatviswa504
Copy link
Contributor

Thank You @dineshchitlangia for the contribution.
I have committed this to the trunk.

@anuengineer
Copy link
Contributor

@bharatviswa504 Thanks for the reviews. @dineshchitlangia Thank you for the contribution. I have committed this to the trunk.

@dineshchitlangia
Copy link
Contributor Author

Thank you @bharatviswa504 and @anuengineer

amahussein pushed a commit to amahussein/hadoop that referenced this pull request Oct 29, 2019
RogPodge pushed a commit to RogPodge/hadoop that referenced this pull request Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants