Matthew Jacobs

Per https://github.com/elastic/elasticsearch-java/blob/main/CONTRIBUTING.md#project-structure: > The `co.elastic.clients.elasticsearch` package and its children are all entirely generated, and the generator is not part of this repository. Because of this, PRs will not work for...

[Function literals with receiver](https://kotlinlang.org/docs/lambdas.html#function-literals-with-receiver) make many parts of the DSL simple and elegant to construct: ``` fun SearchRequest.Builder.queryKt(commands: Query.Builder.() -> Unit): SearchRequest.Builder = query(Query.Builder().apply(commands).build()) fun Query.Builder.boolKt(commands: BoolQuery.Builder.() -> Unit): ObjectBuilder...

Done! In doing so I discovered additional bugs. This was a bit weird to write tests for because there were three things to test: 1) Do individual listeners work on...

Current failing set of tests for JDK 17 all fail in the same way: ``` Distributed Test Run :data-hibernate-jpa:test > Partition 7 in session 1 on localhost-executor-1 > EventsPrePersistSpec test...

Okay, that should do it. Had a minor mishap with a bad set of copied listeners but we're good now. Good catch on the closures!

Static Analysis failed due to a known issue re: Sonarqube (if I recall correctly, the token to use Sonarqube isn't accessible to either me or Github Actions). The GraalVM tests...

Found more bugged listener behavior and added it to the description.

Would it be possible (albeit a breaking change) to resolve this issue by storing the success body in one field and the error body in another field of the `HttpClient`?...

Funny enough, there is a version of `HttpClient#exchange` that takes both a body type and error type: https://github.com/micronaut-projects/micronaut-core/blob/08f3e7aa5ae400c1afc287b3a1dc47e7ecd4249a/http-client/src/main/java/io/micronaut/http/client/netty/DefaultHttpClient.java#L837-L843 so it surprises me that if `exception-on-error-status` is set to false we...

It's not exactly a fix - it's a workaround - because it's possible that the value in your data class might be nullable with a non-null default. In that case,...