Skip to content

RYA-501 Change guava Iterators to Collections - #304

Open
isper3at wants to merge 1 commit into
apache:masterfrom
isper3at:RYA-501_guavaToJava-4.x
Open

RYA-501 Change guava Iterators to Collections#304
isper3at wants to merge 1 commit into
apache:masterfrom
isper3at:RYA-501_guavaToJava-4.x

Conversation

@isper3at

Copy link
Copy Markdown
Contributor

Description

What Changed?

Google changed the Iterators object emptyIterator() visibility.
This can cause versioning issues with anything depending on
a newer version of guava. Using Java's Collections.emptyIterator() instead.

Tests

Coverage?

N/A

Links

Jira

Checklist

  • Code Review
  • Squash Commits

People To Reivew

@ejwhite922
@kchilton2
@pujav65

Google changed the Iterators object emptyIterator() visibility.
This can cause versioning issues with anything depending on
a newer version of guava. Using Java's Collections.emptyIterator() instead.
@asfgit

Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/770/

@isper3at

Copy link
Copy Markdown
ContributorAuthor

Closing until the #302 is in

@isper3atisper3at reopened this Oct 1, 2018
@asfgit

Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/782/

Sign up for freeto 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.

3 participants

@isper3at@asfgit@kchilton2