-
Notifications
You must be signed in to change notification settings - Fork 190
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
[BUG] org.opensearch.client.util.MissingRequiredPropertyException: Missing required property 'OpenSearchVersionInfo.distribution' #573
Comments
What version of the java client are you using? |
Using version |
Feel free to submit the PR to fix this. I think it makes sense. |
In that case, I can open a support ticket to AWS to investigate. |
I created a 2.7 domain and I cannot reproduce this, I see distribution below. What do you see @jhg023?
Update: do you use OpenSearch in compatibility mode with |
Unless AWS OpenSearch comes with that compatibility enabled by default, I never explicitly enabled it. Here's what I see using the same
Seeing as how the |
@jhg023 So yes, this is compatibility mode. When you create a new domain there's a checkbox to enable/disable it. If you don't need this, you can toggle it via API
I'm going to close this, but feel free to keep the conversation open as needed. Docs for compatibility mode: https://opensearch.org/docs/latest/tools/index/#agents-and-ingestion-tools |
@jhg023 It's not the default, but there is a check box to enable compatibility mode on an Amazon OpenSearch Service domain when creating it via the console UI. |
Thanks for letting me know! I created this domain through CloudFormation, so I suspect I may have unintentionally enabled it there, or didn't explicitly opt to disable it. Edit: It looks like the |
I don't get it. I thought the issue was that the server response was causing an exception. I don't think compatibility mode should cause an error. |
The compatibility mode (which if I am not mistaken phased out in 3.0.0) is only need when it is desired that OpenSearch cluster could be accessed by Elasticsearch clients (not OpenSearch clients). |
@reta The @wbeckler The question is whether opensearch-java should be compatible with OpenSearch running in a compatibility mode intended to make it compatible with Elasticsearch clients. |
I believe the answer is no, there's no claims of opensearch-java working with Elasticsearch 7.10. |
Wouldn't it be better if the client could be cross compatible? Maybe with a
client compatibility mode flag as well?
…On Tue, Aug 8, 2023, 11:06 AM Daniel (dB.) Doubrovkine < ***@***.***> wrote:
@wbeckler <https://github.com/wbeckler> The question is whether
opensearch-java should be compatible with OpenSearch running in a
compatibility mode intended to make it compatible with Elasticsearch
clients.
I believe the answer is no, there's no claims of opensearch-java working
with Elasticsearch 7.10.
—
Reply to this email directly, view it on GitHub
<#573 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA5PRLWHJSWMQXPIL2YOROTXUJIZHANCNFSM6AAAAAA2DJ723M>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
We definitely shouldn't to take this route, the compatibility was interim measure, at this point the functionality between ES and OS diverged significantly, each has own ecosystem. |
Faced the same issue today and had to set
This will add the property If you are running a managed version of opensearch like on AWS, you need to contact AWS support to update this setting. |
What is the bug?
Earlier today, I upgraded my AWS OpenSearch cluster from version 2.5 to 2.7, and now I'm seeing the following exception when fetching the cluster information on startup:
How can one reproduce the bug?
OpenSearchClient
:What is the expected behavior?
I wouldn't expect this to result in an exception, but rather print the cluster information even if the distribution is missing.
What is your host/environment?
AWS OpenSearch 2.7 (the latest version AWS supports currently)
Do you have any screenshots?
N/A
Do you have any additional context?
If the fix for this issue is as simple as refactoring the following line, I'd be happy to PR it:
For now, I'm going to avoid fetching the cluster information on startup as a temporary solution.
The text was updated successfully, but these errors were encountered: