Skip to content

fix: put columns into cache after full initialization #132

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 13, 2020
Merged

fix: put columns into cache after full initialization #132

merged 1 commit into from
Aug 13, 2020

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Jul 20, 2020 •
edited
Loading

Put columns into cache after full initialization.

see - influxdata/influxdb-java#685

  • CHANGELOG.md updated
  • Rebased/mergeable
  • Sign CLA (if not already signed)
@bednar bednar added the bug Something isn't working label Jul 20, 2020
@bednar bednar requested a review from rhajek July 20, 2020 11:23
@codecov-commenter
Copy link

codecov-commenter commented Jul 20, 2020 •
edited
Loading

Codecov Report

Merging #132 into master will decrease coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #132 +/- ## ============================================ - Coverage 90.92% 90.86% -0.07%  Complexity 441 441 ============================================ Files 137 137 Lines 5391 5388 -3 Branches 205 206 +1 ============================================ - Hits 4902 4896 -6  - Misses 412 414 +2  - Partials 77 78 +1
Impacted Files Coverage Δ Complexity Δ
...om/influxdb/client/internal/MeasurementMapper.java 89.83% <100.00%> (+1.12%) 0.00 <0.00> (ø)
.../influxdb/client/internal/AbstractWriteClient.java 89.57% <0.00%> (-1.90%) 0.00% <0.00%> (ø%)

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 59061d2...63a4c74. Read the comment docs.
@bednar bednar force-pushed the fix/potential_thread_issue branch from ab51d91 to 63a4c74 Compare August 10, 2020 12:38
@bednar bednar merged commit 524994f into master Aug 13, 2020
@bednar bednar deleted the fix/potential_thread_issue branch August 13, 2020 10:54
@bednar bednar added this to the 1.11.0 milestone Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

3 participants

@bednar @codecov-commenter @rhajek