search

elastic / elasticsearch-hadoop

:elephant: Elasticsearch real-time search and analytics natively integrated with Hadoop
https://www.elastic.co/products/hadoop
Apache License 2.0
1.93k stars 986 forks source link

Fix Spring and conjars for 6.8 #2237

Closed zdunecki closed 3 weeks ago

zdunecki commented 4 weeks ago

It took much time since last update on older version of elasticsearch-hadoop and build stops working. In my company we use Elasticsearch 6.8.x that's why it would be helpful for us to have this fix in this version.

I see some version has same problems but currently I'm focused to fix that in version where our dependency is.

cla-checker-service[bot] commented 4 weeks ago

💚 CLA has been signed

masseyke commented 4 weeks ago

Hi @zdunecki. Could you describe the problem you are solving? It looks like the intention is for this to go into a 6.x es-hadoop release. Unfortunately 6.x is long past end of life and won't get any more releases.

zdunecki commented 3 weeks ago

@masseyke I see. Generally speaking for this version (and not only) the build is not possible due the changes of spring and conjars repository url. It's just a fix for make build successfully.

Since at my company we depdens on Elsticsearch 6.8 and I fix this internally (and it works) I thought it may be helpful for community.

If you do not accept changes anymore then you can close this PR but otherwise tell my how can I make this production ready due to conflitcs and CLA (but I accepted that yesterday).

More precisely we need make some changes inside elasticsearch-hadoop (did by fork) but we have to build it first.

masseyke commented 3 weeks ago

I think what you are doing is probably your best option. Es-hadoop is released in sync with Elasticsearch, and there will almost certainly never be another 6.8.x release (the last bugfix release was more than 2 years ago).

zdunecki commented 3 weeks ago

Sure thing.

Leaving the history of this PR for the community may be helpful. If there will be another 6.8.x release then you can think about that once again.

Currently, I'm closing this PR.