Open Bzil opened 1 week ago
Looks like a bug. Want to write a test and fix it?
FYI, we have recently merged a beginning of a code generator that aims to resolve this entire class of problems (https://github.com/opensearch-project/opensearch-java/pull/366). There are a few things you can do to help:
The combination will ensure that as we switch to the generator we're not introducing regressions.
Hi @dblock I'm not familiar with writing a openapi spec, when I try to run opensearch-api-specification tests all of them are in error. So it's quite weird.
@Bzil i am here to help! Feel free to open issues in that repo. Do read the dev guide, some things require a bit of a setup.
What is the bug?
In class
NodeJvmInfo
at row 94 the fieldusingBundledJdk
is expected as not null but in server side code, this field is nullable https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/monitor/jvm/JvmInfo.java#L235 https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/monitor/jvm/JvmInfo.java#L310How can one reproduce the bug?
Call
_nodes
See if you got response with jvm field like that :Try to parse it, See NPE
What is the expected behavior?
Perhaps make this fiels optional
What is your host/environment?
OS : Fedora Linux 38 Java : OpenJDK 64-Bit Server VM : 17.0.9 Open Search 1.3.13, trying to migrate to 2.14