Skip to content

Conversation

mgaido91
Copy link
Contributor

@mgaido91 mgaido91 commented Dec 4, 2017

What changes were proposed in this pull request?

#19696 replaced the deprecated usages for Date and Waiter, but a few methods were missed. The PR fixes the forgotten deprecated usages.

How was this patch tested?

existing UTs

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

OK but let's make sure doing this in one go next time.

@mgaido91
Copy link
Contributor Author

mgaido91 commented Dec 4, 2017

yes @HyukjinKwon , you are 100% right, sorry for this error.

@SparkQA
Copy link

SparkQA commented Dec 4, 2017

Test build #84425 has finished for PR 19875 at commit 424e471.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Dec 4, 2017

Merged to master

@asfgit asfgit closed this in 3927bb9 Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants