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] bwc test for text chunking processor #702

Merged

Conversation

yuye-aws
Copy link
Member

@yuye-aws yuye-aws commented Apr 23, 2024

Description

Backport PR from #661

Issues Resolved

Backport PR from #661

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed as 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.

* bwc test for text chunking processor

Signed-off-by: yuye-aws <[email protected]>

* spotless apply

Signed-off-by: yuye-aws <[email protected]>

* update changelog

Signed-off-by: yuye-aws <[email protected]>

* spotless apply

Signed-off-by: yuye-aws <[email protected]>

* add test document for restart upgrade

Signed-off-by: yuye-aws <[email protected]>

* rename pipeline configuration file

Signed-off-by: yuye-aws <[email protected]>

* fix pipeline create bug

Signed-off-by: yuye-aws <[email protected]>

* fix pipeline create bug

Signed-off-by: yuye-aws <[email protected]>

* filter tests for lower versions

Signed-off-by: yuye-aws <[email protected]>

* index create in chunking bwc test

Signed-off-by: yuye-aws <[email protected]>

* index create in chunking bwc test

Signed-off-by: yuye-aws <[email protected]>

* index create in chunking bwc test

Signed-off-by: yuye-aws <[email protected]>

* index validate in chunking bwc test

Signed-off-by: yuye-aws <[email protected]>

* filter bwc test for lower version

Signed-off-by: yuye-aws <[email protected]>

* bug fix in document ingestion in text chunking test

Signed-off-by: yuye-aws <[email protected]>

* ensure index creation in text chunking bwc test

Signed-off-by: yuye-aws <[email protected]>

* add comment

Signed-off-by: yuye-aws <[email protected]>

* update index setting

Signed-off-by: yuye-aws <[email protected]>

* update change log

Signed-off-by: yuye-aws <[email protected]>

* update gradle comment format

Signed-off-by: yuye-aws <[email protected]>

* update gradle file format

Signed-off-by: yuye-aws <[email protected]>

* rename bwc test filename

Signed-off-by: yuye-aws <[email protected]>

* update gradle file format

Signed-off-by: yuye-aws <[email protected]>

* update gradle file to filter tests

Signed-off-by: yuye-aws <[email protected]>

* merge method createPipelineProcessorWithoutModelId

Signed-off-by: yuye-aws <[email protected]>

* text chunking processor it: create pipeline method rename

Signed-off-by: yuye-aws <[email protected]>

* fix it failure

Signed-off-by: yuye-aws <[email protected]>

* include index mapping for text chunking index setting

Signed-off-by: yuye-aws <[email protected]>

* update nitpicking

Signed-off-by: yuye-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>
(cherry picked from commit e69752c)
@yuye-aws yuye-aws force-pushed the backport/backport-661-to-2.x branch from c05f6b2 to c937a2a Compare April 23, 2024 02:03
Signed-off-by: yuye-aws <[email protected]>
@vibrantvarun
Copy link
Member

Waiting for this PR on ml-commons to get merged

@yuye-aws
Copy link
Member Author

The ml-commons PR gets merged in main and 2.x branch: opensearch-project/ml-commons#2351

@vibrantvarun vibrantvarun merged commit c6a8537 into opensearch-project:2.x Apr 24, 2024
84 checks passed
@yuye-aws yuye-aws deleted the backport/backport-661-to-2.x branch April 24, 2024 23:09
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.

3 participants