Skip to content

Conversation

@thalurur
Copy link
Contributor

@thalurur thalurur commented May 21, 2021

Issue #, if available:

Description of changes:
Introducing transforms feature to index management - https://github.com/opensearch-project/index-management/blob/main/docs/transform-rfc.md

Most of the code was 1 person reviewed during development

CheckList:
[ X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

thalurur and others added 21 commits March 1, 2021 12:06
* Adding stream tests for transform object

* Added Create/Update API

* Added license text to Create/Update API files

* Updated config to support transforms, additional integ tests for create transform

* Corrected issues noted in review in PR#412

* Updating TransportIndexTransformAction function names

Co-authored-by: Ravi Thaluru <ravi1092@gmail.com>
* Initial GetTransform commit

* Added GetTransform API, unit tests, integ tests

* Ktlint fixes

* Implementing util function, modified missing source behavior, xContentRegistry tweaks

* Updated Transport logic, renamed GetUtils to ScheduledJobUtils and streamlined method signature
* Added Delete Transform API and tests
* Start/Stop API initial commit
Signed-off-by: Ravi Thaluru <ravi1092@gmail.com>
Signed-off-by: Ravi Thaluru <ravi1092@gmail.com>
Signed-off-by: Ravi Thaluru <ravi1092@gmail.com>
Signed-off-by: Ravi Thaluru <ravi1092@gmail.com>
Signed-off-by: Ravi Thaluru <ravi1092@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #17 (1d2c8ea) into main (72018d4) will increase coverage by 0.78%.
The diff coverage is 81.25%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #17      +/-   ##
============================================
+ Coverage     77.31%   78.10%   +0.78%     
- Complexity     1557     1856     +299     
============================================
  Files           201      255      +54     
  Lines          8588     9962    +1374     
  Branches       1326     1506     +180     
============================================
+ Hits           6640     7781    +1141     
- Misses         1250     1366     +116     
- Partials        698      815     +117     
Impacted Files Coverage Δ Complexity Δ
...pensearch/indexmanagement/IndexManagementRunner.kt 55.55% <0.00%> (+5.55%) 4.00 <0.00> (+1.00)
...arch/indexmanagement/rollup/RollupMapperService.kt 58.87% <ø> (ø) 23.00 <0.00> (ø)
...ch/indexmanagement/rollup/RollupMetadataService.kt 75.24% <ø> (ø) 34.00 <0.00> (ø)
...management/rollup/interceptor/RollupInterceptor.kt 68.96% <ø> (ø) 31.00 <0.00> (ø)
...ensearch/indexmanagement/rollup/model/ISMRollup.kt 93.04% <ø> (ø) 27.00 <0.00> (ø)
.../opensearch/indexmanagement/rollup/model/Rollup.kt 89.17% <ø> (ø) 53.00 <0.00> (ø)
...ansform/exceptions/TransformValidationException.kt 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...ensearch/indexmanagement/util/ScheduledJobUtils.kt 26.92% <26.92%> (ø) 0.00 <0.00> (?)
...opensearch/indexmanagement/rollup/RollupIndexer.kt 58.92% <50.00%> (-2.09%) 8.00 <0.00> (ø)
...form/action/index/TransportIndexTransformAction.kt 54.00% <54.00%> (ø) 3.00 <3.00> (?)
... and 121 more

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 72018d4...1d2c8ea. Read the comment docs.

@thalurur thalurur merged commit 106a108 into main May 25, 2021
wuychn pushed a commit to ochprince/index-management that referenced this pull request Mar 16, 2023
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.

4 participants