quarkiverse / quarkus-jnosql

The Quarkus JNoSql Extension adds support for JNoSQL, an implementation of Jakarta NoSQL.
http://www.jnosql.org/
Apache License 2.0
13 stars 4 forks source link

Add KeyValue Couchbase Support #31

Open dearrudam opened 1 year ago

dearrudam commented 1 year ago

Changes

dearrudam commented 1 year ago

@amoscatelli it looks like com.couchbase.client:java-client:3.3.4 doesn't support native compilation...I've tried to solve it by following this doc: https://quarkus.io/version/main/guides/native-reference#i-get-a-analysiserrorparsingerror-when-building-a-native-executable-due-to-an-unresolvedelementexception-what-can-i-do ...

According to this info:

In the unfortunate case where the reference causing the issue is made by a 3rd party library, that you cannot modify, you should consider one of the following:

  1. Use a class/method substitution to remove the said reference.

  2. Add the optional dependency as a non-optional dependency of your project.

The option 1 it's not possible...this com.couchbase.client:java-client:3.3.4 dependency is a required one;

The option 2 says that the com.couchbase.client:java-client:3.3.4 dependency is pointing to many classes that it's not in the classpath... I've tried to add it, but adding the required dependencies arent' a practical process to detect.

Do you think that it's a problem for accepting and merging this PR to the source?

FYI, couchbase also can be used as Document NoSQL database... I'm working on this extension...

amoscatelli commented 1 year ago

I believe is possible to have the coachbase driver working in native mode since there is a quarkus camel extension :

https://camel.apache.org/camel-quarkus/3.0.x/reference/extensions/couchbase.html

Try doing something similar to what I did for ArangoDB

otaviojava commented 9 months ago

@gastaldi could you review/merge this one?

gastaldi commented 9 months ago

@otaviojava I'd defer that to @amoscatelli, as I don't have enough knowledge about this :)

amoscatelli commented 9 months ago

@gastaldi @otaviojava this PR offers no native support.

This needs to be fixed before proceeding.

Maybe you could subscribe and hasten on that issue ...