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

[WIP - Draft]✨ Promote Couchbase to native mode #3365

Closed
wants to merge 1 commit into from

Conversation

zbendhiba
Copy link
Contributor

Fixes #1188

Testing with CI -- Do not merge please

@zbendhiba zbendhiba marked this pull request as draft December 8, 2021 09:35
@zbendhiba zbendhiba changed the title [WIP]✨ Promote Couchbase to native mode [WIP - Draft]✨ Promote Couchbase to native mode Dec 8, 2021
@jamesnetherton
Copy link
Contributor

Are those couchbase config properties mandatory? If so, it'd be a blocker to using the extension in CQ IMO.

@zbendhiba
Copy link
Contributor Author

Are those couchbase config properties mandatory? If so, it'd be a blocker to using the extension in CQ IMO.

yes they are. I'd open an issue then

@zbendhiba
Copy link
Contributor Author

Are those couchbase config properties mandatory? If so, it'd be a blocker to using the extension in CQ IMO.

yes they are. I'd open an issue then

@jamesnetherton it's a more general issue, I guess. It's mandatory just for the fact that the extension is present in the pom.xml. couchbaselabs/quarkus-couchbase#5
It's in alpha stage, so hopefully this would be fixed. The idea of my work is to give it a try and give feedback

@zbendhiba
Copy link
Contributor Author

There is a failure somewhere, but the Couchbase tests are fine. First time, we do have some tests working on JVM mode. And they work also on native mode.
We just need to look for bug fixes and improvements, to be able to use it in the future.
I can close the Draft PR

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

Successfully merging this pull request may close these issues.

Couchbase native support
2 participants