search

geraldhumphries / generator-jhipster-elasticsearch-reindexer

A JHipster module that inserts all your data into Elasticsearch and helps solve mapping and maintenance issues.
Other
45 stars 22 forks source link

Is this plugin going to be updated for jhipster > 6.5? #63

Open nomuna opened 4 years ago

nomuna commented 4 years ago

I tried to use this plugin with an application generated using jhipster 6.5.0. If you have no time or intention to upgrade it, can you suggest an alternative.

Thanks.

ruddell commented 4 years ago

Maintaining a module is a lot of work, so I understand why this project hasn't been updated recently.

As an alternative, you can add the ElasticsearchIndexService.java and ElasticsearchIndexResource.java from this example https://github.com/ruddell/jhipster-examples/commit/7ab7510cd2582c5998d9a8ae1965ec7ba324f882 to at least get a REST endpoint working with v6.x (that example was generated with v6.5.1 but should work for you).

You will have to add the other entities you want reindexed to the reindexAll method here (and add the repository and search repository to the constructor). If you want a UI with a button to trigger reindexing, you'll have to add that yourself (currently accessible for easy calls through the Swagger UI).

erikcvisser commented 4 years ago

Hi @ruddell, that is basically the approach I have taken - just using the Service and Resource file and use the REST endpoint (the frontend didn't work anyhow for my React app). However, I've started encountering compilation errors on machines that have a more recent version than java 1.8. On my local machine, I have installed adoptopenjdk-8 (which works), but the jhipster Dockerfile (used in Gitlab CI/CD) specifies openjdk-11-jdk, which gives the compilation error.

(..)/service/ElasticsearchIndexService.java:[116,32] cannot access com.sun.beans.introspect.PropertyInfo class file for com.sun.beans.introspect.PropertyInfo not found

Line that causes this: return new PropertyDescriptor(field.getName(), entityClass).getReadMethod();

Thanks anyhow for the module!

geraldhumphries commented 4 years ago

I've been meaning to get back to this module and update it to work with the latest version of JHipster. I pushed some commits in January, but didn't make a release because there were some things missing.

There's clearly demand for the module. I'll work on it over the coming week.

erikcvisser commented 4 years ago

Great, appreciated! Let me know if/how I can contribute.

Maybe its also good to test against spring boot 2.3.1 (that will come with next version jhipster), as that has some changes on the data layer as well? And the support for Elasticsearch 7 which will replace 6.8.8 in jhipster.

geraldhumphries commented 4 years ago

I've almost got master working for the latest version of JHipster (6.10.1), but there are a few issues that need to be fixed first.

geraldhumphries commented 4 years ago

Side note: One of the reasons this module is hard to maintain is because it supports every version of JHipster since ~2.27.0, including every minor change for minor JH releases. I may adopt a different strategy to make maintenance easier.

Ebsan commented 4 years ago

I'm looking to help with the issues you mentioned above. I don't have much experience with Travis so I'm not sure about how to help there but looks like the new preferred way is to run the build in debug mode on their servers:

https://stackoverflow.com/a/53354078/2628401

AchrafBelkahla commented 4 years ago

Hi, so any news for using this plugin with jhipster 6.10.2 ?