ml-commons icon indicating copy to clipboard operation
ml-commons copied to clipboard

Switch to async search pipeline processing.

Open austintlee opened this issue 2 years ago • 4 comments

Description

[Describe what this change achieves]

Issues Resolved

issue

Check List

  • [ x] New functionality includes testing.
    • [ x] All tests pass
  • [ ] New functionality has been documented.
    • [ ] New functionality has javadoc added
  • [ 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.

austintlee avatar Oct 06 '23 17:10 austintlee

This will not build cleanly until the async change in the Search Pipeline itself makes it into "core".

austintlee avatar Oct 06 '23 17:10 austintlee

@austintlee is this PR still alive?

HenryL27 avatar Nov 29 '23 22:11 HenryL27

I think this PR can fix the bug https://github.com/opensearch-project/ml-commons/issues/2334. @austintlee Can you rebase and tune this PR ?

ylwu-amzn avatar Apr 18 '24 19:04 ylwu-amzn

@ylwu-amzn Please take a look at a new PR I created to replace this - #2345

austintlee avatar Apr 21 '24 23:04 austintlee

new PR https://github.com/opensearch-project/ml-commons/pull/2345 merged, should we close this PR now ?

ylwu-amzn avatar Jun 11 '24 22:06 ylwu-amzn

Confirmed with @austintlee , close this PR now

ylwu-amzn avatar Jun 12 '24 19:06 ylwu-amzn