Skip to content
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

Moe Sync #3519

Merged
merged 11 commits into from
Jun 30, 2019
Merged

Moe Sync #3519

merged 11 commits into from
Jun 30, 2019

Conversation

ronshapiro
Copy link
Contributor

This code has been reviewed and submitted internally. Feel free to discuss on the PR and we can submit follow-up changes as necessary.

Commits:

Add missing description tags to guava-parent poms.

We were previously apparently inheriting them from sonatype-oss-parent, until we removed that. Releases don't work without a description. "Invalid POM: /com/google/guava/guava-parent/28.0-android/guava-parent-28.0-android.pom: Project description missing"

c417618


Fix typo in 'stoppingCount' variable name

Fixes #3500

c168ba2


Split catch() blocks so that javac doesn't generate a reference to their common supertype, ReflectiveOperationException.

ReflectiveOperationException isn't available on the old versions of Android that we support.

As far as we know, there is no actual problem with these references because no methods are called on them. However, the new version of Animal Sniffer we're updating to will flag them anyway, and I think that avoiding them is the safe thing to do.

Prepares for #3497

15c9a93


Upgrade animal sniffer version to 1.18

Fixes #3497

afdecf7


Document default values of builder setters.

d5dca36


Use saturatedToNanos() in CacheBuilder to avoid overflows.

RELNOTES=Use saturatedToNanos() in CacheBuilder to avoid overflows.

7d04f72


Tweaks to the list of Guava features.

4dbc7c1


Add more @nullables to variadic parameter lists.

RELNOTES=Add more @nullables to variadic parameter lists.

fbae3f4


Halve memory consumption when creating a Bloom filter.

AtomicLongArray(long[]) clones its input, which is a waste when we're passing
in a new array.

RELNOTES=Halve VmPeak when creating a Bloom filter.

f499820


Use LongMath.saturatedAdd/Subtract in CacheStats.

Fixes #3503

RELNOTES=avoid overflows/underflows in CacheStats

9f3d048


Fix the travis build

ba4111c

cgdecker and others added 11 commits June 28, 2019 10:25
We were previously apparently inheriting them from sonatype-oss-parent, until we removed that. Releases don't work without a description. "Invalid POM: /com/google/guava/guava-parent/28.0-android/guava-parent-28.0-android.pom: Project description missing"

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=252733039
Fixes #3500

RELNOTES=n/a

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=252852929
…eir common supertype, ReflectiveOperationException.

ReflectiveOperationException isn't available on the old versions of Android that we support.

As far as we know, there is no actual problem with these references because no methods are called on them. However, the new version of Animal Sniffer we're updating to will flag them anyway, and I think that avoiding them is the safe thing to do.

Prepares for #3497

RELNOTES=n/a

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=252888170
Fixes #3497

RELNOTES=n/a

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=252891687
RELNOTES=n/a

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=252895367
RELNOTES=Use saturatedToNanos() in CacheBuilder to avoid overflows.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=253341058
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=253664020
RELNOTES=Add more @nullables to variadic parameter lists.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=254718073
AtomicLongArray(long[]) clones its input, which is a waste when we're passing
in a new array.

RELNOTES=Halve VmPeak when creating a Bloom filter.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=254770497
Fixes #3503

RELNOTES=avoid overflows/underflows in CacheStats

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=254899180
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=255595656
@ronshapiro ronshapiro added the internal-sync A PR that represents a push from Google's internal repo label Jun 28, 2019
@googlebot
Copy link
Collaborator

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of all the commit author(s), set the cla label to yes (if enabled on your project), and then merge this pull request when appropriate.

ℹ️ Googlers: Go here for more info.

@googlebot
Copy link
Collaborator

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

@ben-manes
Copy link
Contributor

@kluever I think you missed an overflow check in SimpleStatsCounter#snapshot().

@ronshapiro ronshapiro merged commit af57f19 into master Jun 30, 2019
@cpovirk cpovirk deleted the sync-master-2019/06/28 branch August 21, 2019 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes internal-sync A PR that represents a push from Google's internal repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Long overflow in cache statistics
10 participants