Feat: add support for docvalue_fields and stored_fields in TopHitsAggregation

Philippus / elastic4s

🔍 Elasticsearch Scala Client - Reactive, Non Blocking, Type Safe, HTTP Client

Apache License 2.0

1.64k stars 695 forks source link

Closed paulMConstant closed 1 month ago

paulMConstant commented 1 month ago

Hello,

It is currently impossible to my understanding to add "docvalue_fields" to a top hits aggregation. Here's a proposition using the storedFields field.

Thanks

paulMConstant commented 1 month ago

Thank you for the quick review. I fixed the storedFields addition and added docValueFields as done in InnerHit.

Philippus commented 1 month ago

Thank you!

paulMConstant commented 1 month ago

Thank you for the quick merge !

paulMConstant commented 1 month ago

Do you think you could make a release soon ?

Philippus commented 1 month ago

I'd like to include the update to Elasticsearch 8.15.3, which looks to be imminent.