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

[Backport 2.x] Ignore all malformed objects when ignore_malformed is true #4518

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 45c9597 from #4494

Fixes a bug to not fail the entire document when "ignore_malformed" is set to true. Allowing the valid fields
to be indexed and ignore only the malformed fields.

Signed-off-by: Hauck <[email protected]>
(cherry picked from commit 45c9597)
@opensearch-trigger-bot opensearch-trigger-bot bot requested review from a team and reta as code owners September 15, 2022 03:23
@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2022

Codecov Report

Merging #4518 (5e44c84) into 2.x (3612b24) will increase coverage by 0.05%.
The diff coverage is 30.00%.

@@             Coverage Diff              @@
##                2.x    #4518      +/-   ##
============================================
+ Coverage     70.60%   70.66%   +0.05%     
+ Complexity    57145    57131      -14     
============================================
  Files          4585     4585              
  Lines        274508   274512       +4     
  Branches      40229    40232       +3     
============================================
+ Hits         193817   193973     +156     
+ Misses        64487    64274     -213     
- Partials      16204    16265      +61     
Impacted Files Coverage Δ
.../java/org/opensearch/index/mapper/FieldMapper.java 80.74% <30.00%> (-0.71%) ⬇️
.../java/org/opensearch/node/NodeClosedException.java 50.00% <0.00%> (-50.00%) ⬇️
...ch/transport/ReceiveTimeoutTransportException.java 50.00% <0.00%> (-50.00%) ⬇️
...regations/metrics/AbstractHyperLogLogPlusPlus.java 51.72% <0.00%> (-37.94%) ⬇️
...java/org/opensearch/threadpool/ThreadPoolInfo.java 56.25% <0.00%> (-37.50%) ⬇️
...search/aggregations/metrics/ParsedPercentiles.java 44.21% <0.00%> (-33.69%) ⬇️
...search/search/aggregations/pipeline/EwmaModel.java 24.44% <0.00%> (-28.89%) ⬇️
...va/org/opensearch/monitor/process/ProcessInfo.java 68.00% <0.00%> (-24.00%) ⬇️
...search/aggregations/pipeline/HoltWintersModel.java 21.47% <0.00%> (-22.82%) ⬇️
...adcast/BroadcastShardOperationFailedException.java 77.77% <0.00%> (-22.23%) ⬇️
... and 454 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@kotwanikunal
Copy link
Member

@hauck-jvsh Can you please help fix the DCO check by performing a sign off? Instructions here.

@hauck-jvsh
Copy link
Contributor

hauck-jvsh commented Sep 21, 2022

I didn't have write access to edit this branch, so I made another pull request #4566 to fix the DCO.

@hauck-jvsh
Copy link
Contributor

This is already merged and should be closed.

@dblock dblock closed this Sep 27, 2022
Sign up for free to 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.

5 participants