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

2.x: Gradle to use less memory, missed an export command #5436

Merged
merged 1 commit into from
Jun 23, 2017

Conversation

akarnokd
Copy link
Member

Looks like there was a missing "export" keyword in the build options and thus gradle ate all 3 GB memory dancing on the edge of the OOMKiller.

@akarnokd akarnokd changed the title Gradle to use less memory, missed an export command 2.x: Gradle to use less memory, missed an export command Jun 23, 2017
@akarnokd
Copy link
Member Author

Much better:

https://travis-ci.org/ReactiveX/RxJava/builds/246182350#L687

image

@codecov
Copy link

codecov bot commented Jun 23, 2017

Codecov Report

Merging #5436 into 2.x will increase coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #5436      +/-   ##
============================================
+ Coverage     95.98%   96.17%   +0.19%     
  Complexity     5803     5803              
============================================
  Files           631      631              
  Lines         41297    41297              
  Branches       5744     5744              
============================================
+ Hits          39639    39718      +79     
+ Misses          657      605      -52     
+ Partials       1001      974      -27
Impacted Files Coverage Δ Complexity Δ
...vex/internal/operators/single/SingleTakeUntil.java 86.88% <0%> (-6.56%) 2% <0%> (ø)
...tivex/internal/schedulers/InstantPeriodicTask.java 61.11% <0%> (-5.56%) 7% <0%> (-2%)
.../internal/operators/maybe/MaybeTakeUntilMaybe.java 95.91% <0%> (-4.09%) 2% <0%> (ø)
...reactivex/internal/operators/single/SingleAmb.java 96.36% <0%> (-3.64%) 9% <0%> (-1%)
...vex/internal/operators/flowable/FlowableTimer.java 92.85% <0%> (-3.58%) 2% <0%> (ø)
...ternal/operators/observable/ObservablePublish.java 92.1% <0%> (-3.51%) 10% <0%> (ø)
...x/internal/operators/maybe/MaybeSwitchIfEmpty.java 97.22% <0%> (-2.78%) 2% <0%> (ø)
...rnal/operators/flowable/FlowableTakeLastTimed.java 96.29% <0%> (-2.78%) 2% <0%> (ø)
...ava/io/reactivex/internal/util/HalfSerializer.java 97.5% <0%> (-2.5%) 22% <0%> (-1%)
.../io/reactivex/internal/schedulers/IoScheduler.java 92.39% <0%> (-2.18%) 9% <0%> (ø)
... and 42 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8adfa3f...9647aee. Read the comment docs.

@akarnokd akarnokd merged commit 14a83b3 into ReactiveX:2.x Jun 23, 2017
@akarnokd akarnokd deleted the GradleOptsLimitMem branch June 23, 2017 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant