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

Fix failing Elasticsearch 7 tests #20

Open
alexander-schranz opened this issue Feb 18, 2021 · 5 comments
Open

Fix failing Elasticsearch 7 tests #20

alexander-schranz opened this issue Feb 18, 2021 · 5 comments
Labels
enhancement New feature or request

Comments

@alexander-schranz
Copy link
Member

alexander-schranz commented Feb 18, 2021

Currently the Test Setup is created for Elasticsearch 5 which supports multiple Types and other features which where removed in newer Versions.

We did introduce a tests with ES 7 in: #19

They currently will fail because of this kind of features. We would need to investigate here some time to decide which tests are needed for ES7 and which are ES5 only tests.

@alexander-schranz alexander-schranz changed the title Fix failing Elasticsearch 7 Fix failing Elasticsearch 7 tests Feb 18, 2021
@toooni
Copy link

toooni commented Feb 18, 2021

How long do you think you will need to support Elasticsearch 5 in new releases? elasticsearch/elasticsearch 5.x seems to only support PHP ^7.0, which is EOL for almost two years now.

EDIT: Sorry, my fault. ^7.0 means also PHP 7.4 ;) But Elasticsearch 5.6.x is EOL since 2019-03-11.

@alexander-schranz
Copy link
Member Author

alexander-schranz commented Feb 18, 2021

@toooni At current state aslong as it would be possible to support a wide range of elasticsearch versions we will try to support it. It would be more maintenance work to have 2 different versions of the bundle to support and keep up2date with PHP versions and Symfony versions, as I know that there are still Projects using ES5. At current state it seems like there are more projects using ES5 then ES6 and ES7

See the graphics at the bottom with the major version filter:

@alexander-schranz alexander-schranz added the enhancement New feature or request label Feb 18, 2021
@toooni
Copy link

toooni commented Feb 18, 2021

I agree with the idea, but since elasticsearch/elasticsearch for ES5 doesn't support PHP8, there is not much in 5.x to maintain anymore. No new PHP Versions which will support using Elasticsearch 5.x. This ended with PHP7.4.
New features do not make much sense to implement for this version. The only case where changes for 5.x would be needed is bugfixes. But this bundle at the current state is VERY stable IMO.

@alexander-schranz
Copy link
Member Author

Yeah there is currently no active development for features, I think we will decide the supported versions of PHP, Symfony and ES when a new Symfony or ElasticSearch Major Version is released and how much work it would then to support old versions.

@toooni
Copy link

toooni commented Feb 18, 2021

Symfony 6 will arrive in November. PHP 7.4 EOL will be in November too. So, chances are that there won't be PHP7.4 support in Symfony 6. We'll see. I agree that there is no need to rush things right now. I'd recommend to wait with resolving this issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants