Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Elasticsearch 8.0 support #2

Merged
merged 15 commits into from
Feb 27, 2022
Merged

Elasticsearch 8.0 support #2

merged 15 commits into from
Feb 27, 2022

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants