Skip to content

Tanik derived source test#18465

Closed
tanik98 wants to merge 12 commits intoopensearch-project:mainfrom
tanik98:tanik-derived-source-test
Closed

Tanik derived source test#18465
tanik98 wants to merge 12 commits intoopensearch-project:mainfrom
tanik98:tanik-derived-source-test

Conversation

@tanik98
Copy link
Copy Markdown
Contributor

@tanik98 tanik98 commented Jun 9, 2025

Tanik derived source test

tanik98 and others added 10 commits May 8, 2025 15:24
@tanik98
Adding integration of derived source feature across diff paths
7be78c1 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Using index setting instead of mapping field in _source
0d8ad3d 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Modifying IT to include all the supported field type for derived source
f7656ed 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Removing handling of sub keyword field from IT
0656da9 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Adding changelog
762ced7 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Merge branch 'opensearch-project:main' into tanik-derived-source-feat…
1061552 
…ure-integration
@tanik98
Removing interface changes in Engine and addresing comments
ad3fac0 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Fixing unit tests
f956955 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98
Merge branch 'opensearch-project:main' into tanik-derived-source-feat…
01663be 
…ure-integration
@tanik98
Fixing unit tests
6352a70 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2025

❌ Gradle check result for 123b94d: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@tanik98
Adding ITs for reovery flow with derived source
b8a0b73 
Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98 tanik98 force-pushed the tanik-derived-source-test branch from 123b94d to b8a0b73 Compare June 9, 2025 04:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2025

❌ Gradle check result for b8a0b73: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2025

❌ Gradle check result for baeca21: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@tanik98 tanik98 force-pushed the tanik-derived-source-test branch from baeca21 to a9db13e Compare June 9, 2025 07:25
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2025

❌ Gradle check result for a9db13e: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@tanik98 tanik98 force-pushed the tanik-derived-source-test branch from a9db13e to 79b3651 Compare June 9, 2025 08:10
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2025

❌ Gradle check result for 79b3651: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@tanik98
Adding more IT for reindex flow and UTs in IntenalEngine for derived …
7b80644 
…source

Signed-off-by: Tanik Pansuriya <panbhai@amazon.com>
@tanik98 tanik98 force-pushed the tanik-derived-source-test branch from 79b3651 to 7b80644 Compare June 9, 2025 08:51
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2025

✅ Gradle check result for 7b80644: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 9, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 68.90244% with 51 lines in your changes missing coverage. Please review.

Project coverage is 72.51%. Comparing base (3b018ad) to head (7b80644).
Report is 36 commits behind head on main.

Files with missing lines Patch % Lines
...rg/opensearch/index/engine/TranslogLeafReader.java 43.10% 27 Missing and 6 partials ⚠️
...java/org/opensearch/index/mapper/ObjectMapper.java 33.33% 7 Missing and 1 partial ⚠️
...va/org/opensearch/index/engine/InternalEngine.java 68.75% 3 Missing and 2 partials ⚠️
.../main/java/org/opensearch/index/mapper/Mapper.java 0.00% 2 Missing ⚠️
.../org/opensearch/index/mapper/RootObjectMapper.java 80.00% 2 Missing ⚠️
...in/java/org/opensearch/index/shard/IndexShard.java 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##               main   #18465      +/-   ##
============================================
- Coverage     72.52%   72.51%   -0.02%     
- Complexity    67509    67563      +54     
============================================
  Files          5496     5499       +3     
  Lines        311467   311781     +314     
  Branches      45253    45257       +4     
============================================
+ Hits         225891   226075     +184     
- Misses        67180    67307     +127     
- Partials      18396    18399       +3

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Jun 9, 2025
Closed
@tanik98 tanik98 closed this Jun 9, 2025
This was referenced Jun 10, 2025
Open
Open
@tanik98 tanik98 deleted the tanik-derived-source-test branch June 20, 2025 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anasalkouz anasalkouz Awaiting requested review from anasalkouz anasalkouz will be requested when the pull request is marked ready for review anasalkouz is a code owner

@andrross andrross Awaiting requested review from andrross andrross will be requested when the pull request is marked ready for review andrross is a code owner

@ashking94 ashking94 Awaiting requested review from ashking94 ashking94 will be requested when the pull request is marked ready for review ashking94 is a code owner

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar will be requested when the pull request is marked ready for review Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE will be requested when the pull request is marked ready for review CEHENKLE is a code owner

@cwperks cwperks Awaiting requested review from cwperks cwperks will be requested when the pull request is marked ready for review cwperks is a code owner

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis will be requested when the pull request is marked ready for review dbwiddis is a code owner

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna will be requested when the pull request is marked ready for review gbbafna is a code owner

@jed326 jed326 Awaiting requested review from jed326 jed326 will be requested when the pull request is marked ready for review jed326 is a code owner

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal kotwanikunal will be requested when the pull request is marked ready for review kotwanikunal is a code owner

@mch2 mch2 Awaiting requested review from mch2 mch2 will be requested when the pull request is marked ready for review mch2 is a code owner

@msfroh msfroh Awaiting requested review from msfroh msfroh will be requested when the pull request is marked ready for review msfroh is a code owner

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 will be requested when the pull request is marked ready for review owaiskazi19 is a code owner

@reta reta Awaiting requested review from reta reta will be requested when the pull request is marked ready for review reta is a code owner

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159 Rishikesh1159 will be requested when the pull request is marked ready for review Rishikesh1159 is a code owner

@sachinpkale sachinpkale Awaiting requested review from sachinpkale sachinpkale will be requested when the pull request is marked ready for review sachinpkale is a code owner

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli will be requested when the pull request is marked ready for review saratvemulapalli is a code owner

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja will be requested when the pull request is marked ready for review shwetathareja is a code owner

@sohami sohami Awaiting requested review from sohami sohami will be requested when the pull request is marked ready for review sohami is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tanik98