Comments

skek created an issue. See original summary.

it-cru’s picture

Status: Active » Postponed

@skek: I've switched from Elastic 6.x to 7.x and used latest patch from #3008742: [mlt] query does not support [ids] with 8.x-7.x-dev which currently applies without any problems also on 8.x-7.x-dev and 8.x-7.0-alpha3. Then my MLT queries works without any problems with Elastic 7.10.0 yet.

Maybe we could also add patch for 8.x-6.x to 8.x-7.x branch and then check/see what is missing like #3180718: Allow to modify term selection parameters when using More Like This.

sokru’s picture

Title: Port issue 3008742 to 7.x - [mlt] query does not support [ids] » Port issue 3008742 to 8.x-7.x - [mlt] query does not support [ids]
StatusFileSize
new3.85 KB
avpaderno’s picture

Status: Postponed » Needs review
robertoperuzzo’s picture

Patch #3 works for me.

My configuration:

  • "drupal/core": "9.1.5"
  • "drupal/elasticsearch_connector": "^7.0@alpha"
  • "drupal/search_api": "1.19"
renrhaf’s picture

Status: Needs review » Reviewed & tested by the community

Thanks, patch #3 works fine on my setup too :

  • Drupal core : 9.1.9
  • Search API : 8.x-1.19
  • ElasticSearch Connector : 8.x-7.0-alpha3

The only issue I saw was that I had some errors because I was not using token fields for similarity comparison.
In the views similarity contextual filter, maybe it should be good to only show such fields (Field name: Fields for similarity).
But this is an other story.

  • sokru committed 32525977 on 8.x-7.x
    Issue #3175898 by sokru, skek, robertoperuzzo, IT-Cru, Renrhaf: Port...
sokru’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

uberengineer’s picture

Unfortunately no longer applies after the update to elasticsearch_connector 7.0.0-alpha4

renrhaf’s picture

Patch was commited into 7.0.0-alpha4 release :)