red-fenix
red-fenix
> I haven't done a full review yet but I do have some general thoughts on the implementation: > > * This really needs to be implemented for the OpenSearch...
> Would this PR return a Multipolygon in case for queries like "hamburg"? See [the nominatim response](https://nominatim.openstreetmap.org/ui/search.html?q=hamburg). > > Currently photon returns a slightly confusing, but correct extent as only...
> I haven't done a full review yet but I do have some general thoughts on the implementation: > > * This really needs to be implemented for the OpenSearch...
> Sorry for the long delay in responding. > > I've decided to get a release out first before looking into this again. Because of this there is a minor...
> Couple of more comments after trying this out. The OpenSearch version currently doesn't work at all. It already fails to import. I've made some adjustments and have been trying...
So, I've found out that there were multiple issues with my build: 1. The Opensearch implementation is very different from the Elasticsearch implementation (ser/des vs parsing a Searchit), but I've...
> With #855 merged you can now simply add the plugin to `build.gradle` and enable it by adding it to the list in > > https://github.com/komoot/photon/blob/919784fe80680631a56471b14e4648683ed568fe/app/opensearch/src/main/java/de/komoot/photon/Server.java#L36 Thank you so far...
Opensearch [has fixed it](https://github.com/codelibs/opensearch-runner/issues/3) and I've updated the PR. Both Opensearch tests and Es Embedded are 100% working.
> Still not completely finished with a detailed review but I added some comments. > > I currently see two major issues that still need to be solved: > >...
Another thing: the Tests are working for Elastisearch as well as Opensearch. Though, whilst importing data, I get an 'Error during bulk import.' Debugging this from the commandline is very...