Brett Findlay
Brett Findlay
@reta Interesting 👀 let me take a look.
Ok I see whats happening here, I will update the PR shortly. Regarding the changes to the missing property on `Info` I'm going to split it out into its own...
@reta I have updated the GetTasksStatus.response to now return JsonData. This however raises the question, on [Info.status ](https://github.com/opensearch-project/opensearch-java/blob/1d99e45ead2aa86204fceb56c0e64f121d2d784b/java-client/src/main/java/org/opensearch/client/opensearch/tasks/Info.java#L74) and if it should be following the same pattern?
Ah ok @reta if we want to include that change in the same PR I can probably get to it in the next few days. I think I want to...
@reta PR updated to return JsonData for both properties 👍
> > @reta PR updated to return JsonData for both properties 👍 > > @Bfindlay it looks great, thank you, I think the test case we have is a bit...
> > I can take a look at some iTests. If you had any specific scenario ideas let me know. > > We have some very limited coverage in `org.opensearch.client.opensearch.integTest.AbstractNodesIT`...
@reta I have put this back into draft, sorry for the commit spam on this one. After adding the iTest for tasks, it has caused side effects in many other...
@reta I might not be able to finish this one anytime soon just low on time at the moment. Will see how I go but the additional work added as...
We started encountering this on the 2.8.0 client as well. It looks to be because the failures are trying to be deserialized as strings, but they contain an TaskFailure object....