Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

SO-4972: Allow immediate (partial) document updates via scripts #989

Merged
merged 3 commits into from
Mar 2, 2022

Conversation

apeteri
Copy link
Member

@apeteri apeteri commented Feb 28, 2022

No description provided.

@apeteri apeteri requested a review from cmark February 28, 2022 17:04
@apeteri apeteri self-assigned this Feb 28, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #989 (425fc6e) into 8.x (78c49e9) will increase coverage by 0.01%.
The diff coverage is 84.21%.

Impacted file tree graph

@@             Coverage Diff              @@
##                8.x     #989      +/-   ##
============================================
+ Coverage     64.57%   64.58%   +0.01%     
- Complexity    11884    11894      +10     
============================================
  Files          1691     1692       +1     
  Lines         55587    55625      +38     
  Branches       5148     5149       +1     
============================================
+ Hits          35893    35926      +33     
- Misses        17504    17508       +4     
- Partials       2190     2191       +1     
Impacted Files Coverage Δ
.../com/b2international/index/es/client/EsClient.java 61.53% <ø> (ø)
...national/index/revision/DefaultRevisionWriter.java 82.25% <0.00%> (-1.35%) ⬇️
...m/b2international/index/es/admin/EsIndexAdmin.java 81.23% <73.68%> (-0.41%) ⬇️
...al.index/src/com/b2international/index/Update.java 100.00% <100.00%> (ø)
...com/b2international/index/es/EsDocumentWriter.java 70.94% <100.00%> (+0.25%) ⬆️
...ternational/index/es/client/http/EsHttpClient.java 86.53% <100.00%> (+0.53%) ⬆️
...international/index/es/client/tcp/EsTcpClient.java 76.31% <100.00%> (+0.64%) ⬆️
...index/src/com/b2international/index/es/EsNode.java 81.25% <0.00%> (-1.05%) ⬇️
.../snomed/core/ecl/SnomedEclRefinementEvaluator.java 86.21% <0.00%> (+0.27%) ⬆️
.../b2international/index/es/client/EsClientBase.java 65.90% <0.00%> (+1.13%) ⬆️

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 78c49e9...425fc6e. Read the comment docs.

Copy link
Member

@cmark cmark left a comment

Choose a reason for hiding this comment

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

LGTM!

import java.util.Map;

/**
* @since 8.0.1
Copy link
Member

Choose a reason for hiding this comment

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

Change since to 8.1.0, that's the next minor feature release.

@apeteri apeteri merged commit cbdd7e8 into 8.x Mar 2, 2022
@apeteri apeteri deleted the feature/SO-4972_scripted_document_updates branch March 2, 2022 09:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants