Neetika Singhal
Neetika Singhal
@dblock @nhtruong do u mind taking another look at the pr?
> Good progress, close! See below and get all tests to pass. thanks @dblock for reviewing, successful run of tests is dependent on this pr: https://github.com/opensearch-project/opensearch-api-specification/pull/454
> > Good progress, close! See below and get all tests to pass. > > thanks @dblock for reviewing, successful run of tests is dependent on this pr: #454 @dblock...
> > @dblock should we add a higher version on the tests as the feature flag pr(#454) is blocked on the other fields being added to the spec first? thoughts?...
> @neetikasinghal Looks like the OpenSearch PR got merged, want to finish this and add a 2.17 to CI? sure, what would be the @sha256 for 2.17? Also, the spell...
@dblock there would be some more needs for the test to return a status of 200 - e.g. it needs a node with search role configured, else the API would...
> > @dblock there would be some more needs for the test to return a status of 200 - e.g. it needs a node with search role configured, else the...
> > Thats not a bad idea, however these features would be pulled in as well into the main code someday, so there should be an easy way to remove...
> > @dblock to unblock the specs being present in the package, can we merge this change without the tests and i can add follow-up issues to add the setup...
### [Feature]: Support Explain API for KNN ### Overview The explain API in OpenSearch is helpful to the customers to figure out how the relevance score is calculated for every...