8.15.6 - Make public SortOptions.AdditionalPropertyName

[niemyjski]

Elastic.Clients.Elasticsearch version: 8.15.6

Elasticsearch version: 8.15.1

.NET runtime version: 8.x

Operating system version: Any

Description of the problem including expected versus actual behavior:

Previously SortOptions was FieldSort and we used the SortKey in our Foundatio.Parsers visitors (https://github.com/FoundatioFx/Foundatio.Parsers/blob/main/src/Foundatio.Parsers.ElasticQueries/Visitors/GetSortFieldsVisitor.cs#L21). This was converted to SortOptions but now we don't have access to the new property value which exposes the field being sorted on (AdditionalPropertyName)... We have our own apis to build up sorting via fluent ui and translate that to your search descriptors.

Expected behavior

We should be able to see how SortOptions are sorted...

Reference: FoundatioFx/Foundatio.Parsers#84

[flobernd]

Hi @niemyjski,

I don't want to expose the AdditionalPropertyName field for the reasons I stated in the other issue.

However, we can happily add a SortKey property that just reflects the value of the internal AdditionalPropertyName field.

[niemyjski]

That would solve this use case.