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

Document the OS LTR plugin #8074

Closed
wants to merge 56 commits into from

Conversation

epugh
Copy link
Collaborator

@epugh epugh commented Aug 22, 2024

Description

The OpenSearch LTR plugin doesn't have any information in the documentation website.

See this spreadsheet to track the migration effort: https://docs.google.com/spreadsheets/d/1b81JlMCurjhPi-kaGW8eBkiV7s3x4NrtHpyPO3mlghg/edit?gid=0#gid=0

Issues Resolved

Version

2.17? We need to update the LTR plugin before this get's merged, it's very out of date.

See opensearch-project/opensearch-learning-to-rank-base#38 for the issue tracking the work to get a updated plugin released.

Checklist

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

Copy link

Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged.

Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer.

When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review.

@epugh
Copy link
Collaborator Author

epugh commented Aug 23, 2024

@epugh epugh force-pushed the introduce_ltr_plugin branch from 08d0af6 to 02d7b58 Compare August 23, 2024 13:49
@epugh epugh marked this pull request as draft August 23, 2024 13:49
@epugh epugh force-pushed the introduce_ltr_plugin branch 2 times, most recently from 7e3d176 to b629ea8 Compare August 23, 2024 17:37
@epugh epugh changed the title Introduce LTR plugin Document the OS LTR plugin Aug 23, 2024
@kolchfa-aws kolchfa-aws added the 1 - Backlog - DEV Developer assigned to issue is responsible for creating PR. label Aug 30, 2024
jzonthemtn added a commit to jzonthemtn/opensearch-learning-to-rank-base that referenced this pull request Sep 5, 2024
jzonthemtn added a commit to jzonthemtn/opensearch-learning-to-rank-base that referenced this pull request Sep 5, 2024
Naarcha-AWS and others added 26 commits September 12, 2024 10:40
* Add common operations section to User Guide.

Signed-off-by: Archer <[email protected]>

* Fix link

Signed-off-by: Archer <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Archer <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Update known limitations for kNN based indexes

Signed-off-by: Kunal Kotwani <[email protected]>

* Update _tuning-your-cluster/availability-and-recovery/snapshots/searchable_snapshot.md

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

---------

Signed-off-by: Kunal Kotwani <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
…ject#8113)

* added full file paths for security config files
Signed-off-by: [email protected] <[email protected]>

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

* added full file paths for security config files
Signed-off-by: [email protected] <[email protected]>

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

# Conflicts:
#	_security/configuration/yaml.md

* small edits to full file paths for security config files
Signed-off-by: [email protected] <[email protected]>

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

* updates to file paths following tech review
Signed-off-by: [email protected] <[email protected]>

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

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Take into account previous changes

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: [email protected] <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
The original Data Prepper roadmap was a project linked to the repository. GitHub has been removing these and migrated it to a project linked to the organization instead. With this change on GitHub, the old URL became invalid and now shows a 404 page.

Now commit is updating all occurrences of the old URL to use the new URL.

Signed-off-by: David Venable <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Delete graphs and copy edit

Signed-off-by: Melissa Vagi <[email protected]>

* Delete graphs and copy edit

Signed-off-by: Melissa Vagi <[email protected]>

* Delete graphs and copy edit

Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* user accounts manipulation audit example

Signed-off-by: jazzl0ver <[email protected]>

* user accounts manipulation audit example

Signed-off-by: jazzl0ver <[email protected]>

* user accounts manipulation audit example

Signed-off-by: jazzl0ver <[email protected]>

* Update _security/audit-logs/index.md

Co-authored-by: Craig Perkins <[email protected]>
Signed-off-by: jazzl0ver <[email protected]>

* Update _security/audit-logs/index.md

Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: jazzl0ver <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Co-authored-by: Craig Perkins <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
…t#8198)

* Fix heading levels in geoshape query documentation

Signed-off-by: Fanit Kolchina <[email protected]>

* One more

Signed-off-by: Fanit Kolchina <[email protected]>

---------

Signed-off-by: Fanit Kolchina <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
Signed-off-by: Fanit Kolchina <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Add documentation for max_number_processors

Signed-off-by: Rai <[email protected]>

* Refined the documentation

Signed-off-by: Rai <[email protected]>

* Doc review

Signed-off-by: Melissa Vagi <[email protected]>

* Update _ingest-pipelines/processors/index-processors.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>

* Update _ingest-pipelines/processors/index-processors.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: Rai <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Rai <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Doc update for concurrent search

Signed-off-by: Ganesh Ramadurai <[email protected]>

* Apply suggestions from code review

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Ganesh Krishna Ramadurai <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: Ganesh Ramadurai <[email protected]>
Signed-off-by: Ganesh Krishna Ramadurai <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Add new settings for remote publication

Signed-off-by: Sooraj Sinha <[email protected]>

* Update remote-cluster-state.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* remove redundant lines

Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Sooraj Sinha <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
…#8207)

* Add documentation changes for shallow snapshot

Signed-off-by: Anshu Agarwal <[email protected]>

* Update create-repository.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Update snapshot-interoperability.md

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Anshu Agarwal <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Co-authored-by: Anshu Agarwal <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* adding asciifolding token filter page opensearch-project#7873

Signed-off-by: AntonEliatra <[email protected]>

* updating the naming

Signed-off-by: AntonEliatra <[email protected]>

* updating as per PR comments

Signed-off-by: AntonEliatra <[email protected]>

* updating the heading

Signed-off-by: AntonEliatra <[email protected]>

* Updating details as per comments

Signed-off-by: AntonEliatra <[email protected]>

* Updating details as per comments

Signed-off-by: AntonEliatra <[email protected]>

* Updating details as per comments

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* updating as per comments

Signed-off-by: Anton Rubin <[email protected]>

* Apply suggestions from code review

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

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>

* Update asciifolding.md

Signed-off-by: AntonEliatra <[email protected]>

---------

Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: Anton Rubin <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
…search-project#8196)

* Create documentation for snapshots with hashed prefix path type

Signed-off-by: Ashish Singh <[email protected]>

* Add documentation on new cluster settings for fixed prefix

Signed-off-by: Ashish Singh <[email protected]>

* Update create-repository.md

* Update create-repository.md

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Signed-off-by: Naarcha-AWS <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>

---------

Signed-off-by: Ashish Singh <[email protected]>
Signed-off-by: Naarcha-AWS <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Adding new cluster search setting docs

Signed-off-by: Harsha Vamsi Kalluri <[email protected]>

* Update _install-and-configure/configuring-opensearch/search-settings.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Harsha Vamsi Kalluri <[email protected]>

---------

Signed-off-by: Harsha Vamsi Kalluri <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Grouping Top N queries documentation

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Fix dead links

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Fix dead link

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Fix dead links

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Address reviewdog comments

Signed-off-by: Siddhant Deshmukh <[email protected]>

* reviewdog fix

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Doc review

Signed-off-by: Fanit Kolchina <[email protected]>

* Add table

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Table review and added ability to collapse the response

Signed-off-by: Fanit Kolchina <[email protected]>

* More explanation to a couple of parameters

Signed-off-by: Fanit Kolchina <[email protected]>

* Typo fix

Signed-off-by: Fanit Kolchina <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>

* Editorial comment

Signed-off-by: Fanit Kolchina <[email protected]>

* Update _observing-your-data/query-insights/grouping-top-n-queries.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: Siddhant Deshmukh <[email protected]>
Signed-off-by: Fanit Kolchina <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: Fanit Kolchina <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
…t#8172)

* Document reprovision param for Update Workflow API

Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

---------

Signed-off-by: Daniel Widdis <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
* Update GET top N api documentation

Signed-off-by: David Zane <[email protected]>

* Update _observing-your-data/query-insights/top-n-queries.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: David Zane <[email protected]>

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: David Zane <[email protected]>
Signed-off-by: David Zane <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
opensearch-project#8127)

* Update doc for adding new param in cat shards action for cancellation support

Signed-off-by: Somesh Gupta <[email protected]>

* Fixed comment

Signed-off-by: Somesh Gupta <[email protected]>

---------

Signed-off-by: Somesh Gupta <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
opensearch-project#8118)

* add doc: skip_validating_missing_parameters

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

* Doc review

Signed-off-by: Fanit Kolchina <[email protected]>

* update: make both description consistent

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

* Apply suggestions from code review

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>
Signed-off-by: Fanit Kolchina <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: Fanit Kolchina <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
opensearch-project#8171)

* Document use_case param for Create Workflow API, link to existing docs

Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Update _automating-configurations/api/create-workflow.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>

* Resolve merge conflicts

Signed-off-by: Fanit Kolchina <[email protected]>

---------

Signed-off-by: Daniel Widdis <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Signed-off-by: Fanit Kolchina <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Co-authored-by: Fanit Kolchina <[email protected]>
Signed-off-by: Eric Pugh <[email protected]>
@epugh epugh force-pushed the introduce_ltr_plugin branch from 3fc4cf0 to 444a813 Compare September 12, 2024 14:41
@epugh epugh mentioned this pull request Sep 18, 2024
@epugh
Copy link
Collaborator Author

epugh commented Sep 18, 2024

Closing in favour of #8336

@epugh epugh closed this Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - Backlog - DEV Developer assigned to issue is responsible for creating PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.