ES 5.1? about plastic HOT 13 CLOSED

So after some discussion with the elasticsearch/elasticsearch author, he was willing to add a new library to the class to help out our cause of keeping support for 5.x. Take a look:

elastic/elasticsearch-php#534 (comment)

from plastic.

@sleimanx2 What needs doing on this? Is the work on the develop branch current, or is it out of date? I note that it is currently 32 commits behind master.

from plastic.

@sleimanx2 Any updates on this? We'd like to use it on our Laravel 5.4, ES 5.4 and PHP 7 site.

from plastic.

Please check v0.5.0 🥂

from plastic.

perfect added a develop branch for ES 5 support

from plastic.

I think this needs to be reopened and discussed a little more.

Right now you're wanting to include support for Laravel 5.2.*|5.3.*. Well, 5.4.* is now a thing.

Also, the underlying library of elasticsearch/elasticsearch is going to require PHP ^7.0 as they require ext-json >= 1.3.7. PHP 5.6.x ships with ext-json 1.2.1.

So if we want to support the elasticsearch/elasticsearch 5.x the minimum PHP version Plastic will need to support is 7.0. This cuts out all current Laravel versions as the framework only requires 5.6.4+.

Laravel 5.5 will require PHP 7.0+. So until Laravel requires 7.0 I would halt development and support on ES 5.

Once Laravel 5.5 is released we can change the composer.json to the following:

  "require": {
    "php": ">=7.0",
    "elasticsearch/elasticsearch": "~5.1",
    "ongr/elasticsearch-dsl": "~5.0"
    "illuminate/container": "5.5.*",
    "illuminate/contracts": "5.5.*",
    "illuminate/console": "5.5.*",
    "illuminate/pagination": "5.5.*",
    "illuminate/support": "5.5.*",
    "illuminate/database": "5.5.*",
  },

from plastic.

@ellisio I understand where you come from but what solution can we arrive at for people running PHP7 and wanting ES 5? Should we have a new major version of this library for these latest versions?

from plastic.

@ellisio how should we use this now?

from plastic.

@sleimanx2 @ellisio The ES library has been updated to 5.1.3 to support this - what are the next steps?

from plastic.

@ellisio @sleimanx2 any further update on how to get this to be compatible with ES 5.X?? I'm currently working on a large project involving laravel 5.4 with PHP 7.0.2 (Which is required for some of the processing within laravel) and would like to require this package with the latest ES.

from plastic.

@ellisio @sleimanx2 I'm also very interested having this package compatible with ES 5.X and PHP 7. Could we create a separate branch for that until Laravel 5.5 in July?

from plastic.

the issue is that the test are not green anymore due to the following update

Since Elasticsearch 5.0 the support for top level filters was dropped. The same functionality is now supported via BoolQuery

I'll work on fixing the issue soon.

from plastic.

Hey @sleimanx2 is there anything the community can do to help move this?

from plastic.