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.10] [Segment Replication] Support realtime TermVector requests with Segment Replication #9936

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport caf4c80 from #9585.

…nt Replication (#9585)

* support realtime TermVector and MultiTermVector requests with segment replication.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

* Fix TermVector requests with segrep.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

* Refacotring.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

* Address comments on PR.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

---------

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>
Signed-off-by: Rishikesh Pasham <62345295+Rishikesh1159@users.noreply.github.com>
(cherry picked from commit caf4c80)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2023

Compatibility status:

Checks if related components are compatible with change 1ffd1a4

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/performance-analyzer-rca.git]

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.snapshots.CloneSnapshotIT.testCloneAfterRepoShallowSettingDisabled
      1 org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=pit/10_basic/Delete all}

@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #9936 (1ffd1a4) into 2.10 (6729882) will decrease coverage by 0.03%.
Report is 1 commits behind head on 2.10.
The diff coverage is 82.35%.

@@             Coverage Diff              @@
##               2.10    #9936      +/-   ##
============================================
- Coverage     70.85%   70.83%   -0.03%     
+ Complexity    58253    58226      -27     
============================================
  Files          4817     4817              
  Lines        275286   275314      +28     
  Branches      40445    40456      +11     
============================================
- Hits         195066   195030      -36     
- Misses        63496    63637     +141     
+ Partials      16724    16647      -77     
Files Changed Coverage Δ
...action/termvectors/TransportTermVectorsAction.java 51.35% <33.33%> (-3.20%) ⬇️
.../src/main/java/org/opensearch/common/Rounding.java 85.02% <92.85%> (+0.37%) ⬆️

... and 453 files with indirect coverage changes

@Rishikesh1159 Rishikesh1159 merged commit dcb43c8 into 2.10 Sep 8, 2023
@github-actions github-actions bot deleted the backport/backport-9585-to-2.10 branch September 8, 2023 18:47
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.

2 participants