Skip to content

Conversation

@hkulekci
Copy link
Owner

@hkulekci hkulekci commented Feb 16, 2022

Here the list of changes :

Aggregation

  • Percentiles aggregation: disallow specifying same percentile values twice #52257 (issue: #51871)
  • Remove Adjacency_matrix setting #46327 (issues: #46257, #46324)
  • Remove MovingAverage pipeline aggregation #39328
  • Remove deprecated _time and _term sort orders #39450
  • Remove deprecated date histo interval #75000

Geo

  • Disallow creating geo_shape mappings with deprecated parameters #70850 (issue: #32039)
  • Remove bounding box query type parameter #74536

Infra REST API

  • Remove content type required setting #61043
  • Allow parsing Content-Type and Accept headers with version #61427

Search

@matchish
Copy link

I see that phpunit have to be updated. Code coverage not supported with php8 by this version

@hkulekci
Copy link
Owner Author

👍 Let me check it

@hkulekci
Copy link
Owner Author

Interestingly, tests passed for both versions as I tried locally. For now, with PHP 8.0.9, I tried PHPUnit 9.5.13 and 8.5.23 with coverage flag and both are working without any error.

@hkulekci hkulekci marked this pull request as ready for review February 17, 2022 14:13
@matchish
Copy link

Looks great to me)

@hkulekci
Copy link
Owner Author

I could not understand what I need to do related withindices_boost parameter. I think, no need to do any changes.

@matchish
Copy link

Could you point where described breaking changes in es8 for this parameter?

@hkulekci
Copy link
Owner Author

Could you point where described breaking changes in es8 for this parameter?

Inside the search part :

  • Remove the object format for indices_boost #55078

https://www.elastic.co/guide/en/elasticsearch/reference/master/release-notes-8.0.0.html

@hkulekci
Copy link
Owner Author

I think we don't do anything. I will try to put a functional test for it. There is a parameter inside the Search class of this library, but we do not try to format the parameter's value. So, I think it is not this library's responsibility.

@matchish
Copy link

Yeah you're right. We don't check format of indices boosts so no changes.

@matchish
Copy link

We could introduce interface for the parameter, but looks it's not the way of this library

@hkulekci
Copy link
Owner Author

Okay, here is the latest situation. I fetched the list from the release notes of the Elasticsearch page, and I checked the bullets one by one. I close some of them directly without changing the code because of their disinterests. Lastly, we have one issue now is related to sorting with nested fields.

Remove deprecated sort options: nested_path and nested_filter elastic/elasticsearch#42809 (issue: elastic/elasticsearch#27098)

I had not used it before. For this reason, I will investigate a little bit what the changes are. Then we can go through the PR to the main repository.

Thanks.

@hkulekci
Copy link
Owner Author

Okay so, as I see, nested-related deprecation changes are already done at 7.x version.
https://github.com/ongr-io/ElasticsearchDSL/pull/267/files

@hkulekci hkulekci merged commit cac247d into 8.x Feb 27, 2022
@hkulekci
Copy link
Owner Author

hkulekci commented Feb 27, 2022

Before :

POST /_search { "query" : { "term" : { "product" : "chocolate" } }, "sort" : [ { "offer.price" : { "mode" : "avg", "order" : "asc", "nested_path" : "offer", "nested_filter" : { "term" : { "offer.color" : "blue" } } } } ] } 

After :

POST /_search { "query" : { "term" : { "product" : "chocolate" } }, "sort" : [ { "offer.price" : { "mode" : "avg", "order" : "asc", "nested": { "path": "offer", "filter": { "term" : { "offer.color" : "blue" } } } } } ] } 
@hkulekci hkulekci deleted the elasticsearch-8.0-support branch February 27, 2022 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants