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

Added release notes for 2.14.0 release #4707

Merged
merged 1 commit into from
May 14, 2024

Conversation

rishabh6788
Copy link
Collaborator

Description

Added release notes for 2.14.0 release.

Issues Resolved

#4562

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.

@gaiksaya
Copy link
Member

Hi @rishabh6788
Can you please make first letter of each bullet captial and replace OpenSearch ML Common with OpenSearch ML Commons?
Thanks!

@rishabh6788
Copy link
Collaborator Author

Hi @rishabh6788 Can you please make first letter of each bullet captial and replace OpenSearch ML Common with OpenSearch ML Commons? Thanks!

Updated.

Copy link
Member

@gaiksaya gaiksaya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nit. Looks good overall.

### Opensearch Reporting


* T enancy access control ([#992](https://github.com/opensearch-project/reporting/pull/992))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* T enancy access control ([#992](https://github.com/opensearch-project/reporting/pull/992))
* Tenancy access control ([#992](https://github.com/opensearch-project/reporting/pull/992))

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack!

Copy link
Member

@dbwiddis dbwiddis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except for the whitespace.

Comment on lines 344 to 345


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra whitespace here that seems to make the spacing different than for everything else

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack!

### Opensearch Neural Search


* Support k-NN radial search parameters in neural search([#697](https://github.com/opensearch-project/neural-search/pull/697))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missed a space after neural search

Comment on lines 124 to 128
* Fix filter fields, adding geo point and date\_nanos (#285) (#286)
* Change ad plugin jar dependency (#288)
* Remove logic about replace quota for finetuning model (#289) (#291)
* Move search index tool to ml-commons repo (#297)
* Move visualization tool to ml-commons (#296) (#298)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are missing the actual link.
They are using reference link of skills repo instead.

Comment on lines 133 to 136

* Add k-NN clear cache api [#740](https://github.com/opensearch-project/k-NN/pull/740)
* Support radial search in k-NN plugin [#1617](https://github.com/opensearch-project/k-NN/pull/1617)
* Support filter and nested field in faiss engine radial search [#1652](https://github.com/opensearch-project/k-NN/pull/1652)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing () around PR#

### Opensearch Observability


* Tenancy access control ([#1821]https://github.com/opensearch-project/observability/pull/1821)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link missing ending )

Comment on lines 239 to 242
* Make the HitQueue size more appropriate for exact search [#1549](https://github.com/opensearch-project/k-NN/pull/1549)
* Implement the Streaming Feature to stream vectors from Java to JNI layer to enable creation of larger segments for vector indices [#1604](https://github.com/opensearch-project/k-NN/pull/1604)
* Remove unnecessary toString conversion of vector field and added some minor optimization in KNNCodec [1613](https://github.com/opensearch-project/k-NN/pull/1613)
* Serialize all models into cluster metadata [#1499](https://github.com/opensearch-project/k-NN/pull/1499)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing ()

Comment on lines 399 to 402
* Adding max http response string length as a setting, and capping http response string based on that setting (#[876](https://github.com/opensearch-project/notifications/pull/876))
* Adding Max HTTP Response Size IT (#[901](https://github.com/opensearch-project/notifications/pull/901)) (#[909](https://github.com/opensearch-project/notifications/pull/909))
* Replacing spi.utils isHostInDenyList with core.utils isHostInDenyList (#[904](https://github.com/opensearch-project/notifications/pull/904)) (#[913](https://github.com/opensearch-project/notifications/pull/913))
* Upgrade AWS version for SDKs to 1.12.687 (#[884](https://github.com/opensearch-project/notifications/pull/884)) (#[887](https://github.com/opensearch-project/notifications/pull/887))
Copy link
Member

@peterzhuamazon peterzhuamazon May 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# was not part of the link

Comment on lines 450 to 451
* Add stored fields for knn\_vector type [#1630](https://github.com/opensearch-project/k-NN/pull/1630)
* Enable script score to work with model based indices [#1649](https://github.com/opensearch-project/k-NN/pull/1649)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing ()

Comment on lines 522 to 524
* Add micro-benchmark module in k-NN plugin for benchmark streaming vectors to JNI layer functionality. [#1583](https://github.com/opensearch-project/k-NN/pull/1583)
* Add arm64 check when SIMD is disabled [#1618](https://github.com/opensearch-project/k-NN/pull/1618)
* Skip rebuild from scratch after cmake is run [#1636](https://github.com/opensearch-project/k-NN/pull/1636)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing ()

### Opensearch Notifications


* Add 2.14.0 release notes (#[915](https://github.com/opensearch-project/notifications/pull/915))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing # in link

### Opensearch Skills


* Increment byte-buddy version to 1.14.9 (#288)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing actual link

Signed-off-by: Rishabh Singh <sngri@amazon.com>
Copy link
Member

@peterzhuamazon peterzhuamazon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

@peterzhuamazon peterzhuamazon merged commit d59a2bf into opensearch-project:main May 14, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants