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

Add getter for path field in NestedQueryBuilder #4609

Closed
psmagin opened this issue Sep 27, 2022 · 2 comments
Closed

Add getter for path field in NestedQueryBuilder #4609

psmagin opened this issue Sep 27, 2022 · 2 comments
Labels
enhancement Enhancement or improvement to existing feature or request Indexing & Search untriaged

Comments

@psmagin
Copy link
Contributor

psmagin commented Sep 27, 2022

Is your feature request related to a problem? Please describe.
The only way now to get path from NestedQueryBuilder is a reflection

@psmagin psmagin added enhancement Enhancement or improvement to existing feature or request untriaged labels Sep 27, 2022
@saratvemulapalli
Copy link
Member

@psmagin thanks for opening up the issue. Trying to understand, what is the use case of using path?
Also on a side note, could you contribute the change? (It should be pretty straight forward)[1]

[1] https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/index/query/NestedQueryBuilder.java

@psmagin
Copy link
Contributor Author

psmagin commented Sep 29, 2022

@saratvemulapalli, it could be useful for creating some complicated queries, the path could be used as a key field to combine nested queries in one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request Indexing & Search untriaged
Projects
None yet
Development

No branches or pull requests

2 participants