#n1ql.bucket returns the collection in v5

[babltiga]

Based on the answers from #1536 and #1445 it seems that the #n1ql.bucket should always return bucket name starting from version 5, but still even in the latest versions, it returns the collection name.

I'm asking about this line https://github.com/spring-projects/spring-data-couchbase/blob/f41d4b94e07923fec2f75ce71c51ea7f376112d0/src/main/java/org/springframework/data/couchbase/repository/query/StringBasedN1qlQueryParser.java#L169C30-L169C30

Should this be fixed? Or was the change postponed for the next major version?

[mikereiche]

It looks like the second constructor has the correct call at line 197, but the first constructor has the incorrect call at line 169.

That plus StringN1qlQueryCreatorTests.spelTests would need to be changed. I'm trying to figure out if it was left like that. intentionally or if by accident/error.

[babltiga]

OK, if you think it should be changed, I'm ready to create a PR )

[mikereiche]

Yes. Submit the PR and I'll approve it. It will change the behavior mid-version, but it's a bug-fix Be sure to also fix the test case I indicated.

I'm not sure if you're already running the integration tests against a real cb server, but to do that edit src/test/resources/integration.properties to point to a real server before running mvn integration-test.

mine looks like this -

# Couchbase Integration-Test Default Properties
# If set to false, it is assumed that the host is managing the cluster and
# as a result no containers or anything will be spun up.
# Options: containerized, mocked, unmanaged
cluster.type=unmanaged
cluster.unmanaged.seed=127.0.0.1:8091
cluster.adminUsername=Administrator
cluster.adminPassword=password
cluster.unmanaged.numReplicas=0