Skip to content

Commit e85b785

Browse files
committed
Eliminate split packages. (#1912)
Closes #1906.
1 parent 268411c commit e85b785

File tree

6 files changed

+26
-50
lines changed

6 files changed

+26
-50
lines changed

src/main/java/com/couchbase/client/java/transactions/AttemptContextReactiveAccessor.java

-34
This file was deleted.

src/main/java/org/springframework/data/couchbase/core/ReactiveFindByQueryOperationSupport.java

+21-11
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
package org.springframework.data.couchbase.core;
1717

1818
import com.couchbase.client.core.api.query.CoreQueryContext;
19+
import com.couchbase.client.core.api.query.CoreQueryOptions;
1920
import reactor.core.publisher.Flux;
2021
import reactor.core.publisher.Mono;
2122

@@ -28,13 +29,11 @@
2829
import org.springframework.data.couchbase.core.support.TemplateUtils;
2930
import org.springframework.util.Assert;
3031

31-
import com.couchbase.client.core.io.CollectionIdentifier;
3232
import com.couchbase.client.java.ReactiveScope;
3333
import com.couchbase.client.java.codec.JsonSerializer;
3434
import com.couchbase.client.java.query.QueryOptions;
3535
import com.couchbase.client.java.query.QueryScanConsistency;
3636
import com.couchbase.client.java.query.ReactiveQueryResult;
37-
import com.couchbase.client.java.transactions.AttemptContextReactiveAccessor;
3837
import com.couchbase.client.java.transactions.TransactionQueryOptions;
3938
import com.couchbase.client.java.transactions.TransactionQueryResult;
4039

@@ -77,9 +76,9 @@ static class ReactiveFindByQuerySupport<T> implements ReactiveFindByQuery<T> {
7776
private final ReactiveTemplateSupport support;
7877

7978
ReactiveFindByQuerySupport(final ReactiveCouchbaseTemplate template, final Class<?> domainType,
80-
final Class<T> returnType, final Query query, final QueryScanConsistency scanConsistency, final String scope,
81-
final String collection, final QueryOptions options, final String[] distinctFields, final String[] fields,
82-
final ReactiveTemplateSupport support) {
79+
final Class<T> returnType, final Query query, final QueryScanConsistency scanConsistency,
80+
final String scope, final String collection, final QueryOptions options, final String[] distinctFields,
81+
final String[] fields, final ReactiveTemplateSupport support) {
8382
Assert.notNull(domainType, "domainType must not be null!");
8483
Assert.notNull(returnType, "returnType must not be null!");
8584
this.template = template;
@@ -192,10 +191,16 @@ public Flux<T> all() {
192191
return pArgs.getScope() == null ? clientFactory.getCluster().reactive().query(statement, opts)
193192
: rs.query(statement, opts);
194193
} else {
195-
TransactionQueryOptions opts = buildTransactionOptions(pArgs.getOptions());
194+
TransactionQueryOptions options = buildTransactionOptions(pArgs.getOptions());
196195
JsonSerializer jSer = clientFactory.getCluster().environment().jsonSerializer();
197-
return AttemptContextReactiveAccessor.createReactiveTransactionAttemptContext(s.get().getCore(), jSer)
198-
.query(OptionsBuilder.queryContext(pArgs.getScope(), pArgs.getCollection(), rs.bucketName()) == null ? null : rs, statement, opts);
196+
CoreQueryOptions opts = options != null ? options.builder().build() : null;
197+
return s.get().getCore()
198+
.queryBlocking(statement,
199+
pArgs.getScope() == null ? null
200+
: CoreQueryContext.of(rs.bucketName(), pArgs.getScope()),
201+
opts, false)
202+
.map(response -> new TransactionQueryResult(response, jSer));
203+
199204
}
200205
});
201206

@@ -255,10 +260,15 @@ public Mono<Long> count() {
255260
return pArgs.getScope() == null ? clientFactory.getCluster().reactive().query(statement, opts)
256261
: rs.query(statement, opts);
257262
} else {
258-
TransactionQueryOptions opts = buildTransactionOptions(pArgs.getOptions());
263+
TransactionQueryOptions options = buildTransactionOptions(pArgs.getOptions());
259264
JsonSerializer jSer = clientFactory.getCluster().environment().jsonSerializer();
260-
return AttemptContextReactiveAccessor.createReactiveTransactionAttemptContext(s.get().getCore(), jSer)
261-
.query(OptionsBuilder.queryContext(pArgs.getScope(), pArgs.getCollection(), rs.bucketName()) == null ? null : rs, statement, opts);
265+
CoreQueryOptions opts = options != null ? options.builder().build() : null;
266+
return s.get().getCore()
267+
.queryBlocking(statement,
268+
pArgs.getScope() == null ? null
269+
: CoreQueryContext.of(rs.bucketName(), pArgs.getScope()),
270+
opts, false)
271+
.map(response -> new TransactionQueryResult(response, jSer));
262272
}
263273
});
264274

src/main/java/com/querydsl/couchbase/document/AbstractCouchbaseQueryDSL.java renamed to src/main/java/org/springframework/data/couchbase/querydsl/document/AbstractCouchbaseQueryDSL.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
* See the License for the specific language governing permissions and
1414
* limitations under the License.
1515
*/
16-
package com.querydsl.couchbase.document;
16+
package org.springframework.data.couchbase.querydsl.document;
1717

1818
import java.util.Collection;
1919
import java.util.HashMap;

src/main/java/com/querydsl/couchbase/document/CouchbaseDocumentSerializer.java renamed to src/main/java/org/springframework/data/couchbase/querydsl/document/CouchbaseDocumentSerializer.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
* See the License for the specific language governing permissions and
1414
* limitations under the License.
1515
*/
16-
package com.querydsl.couchbase.document;
16+
package org.springframework.data.couchbase.querydsl.document;
1717

1818
import java.util.Collection;
1919
import java.util.List;

src/main/java/org/springframework/data/couchbase/repository/support/SpringDataCouchbaseQuerySupport.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@
2424

2525
import com.querydsl.core.support.QueryMixin;
2626
import com.querydsl.core.types.OrderSpecifier;
27-
import com.querydsl.couchbase.document.AbstractCouchbaseQueryDSL;
28-
import com.querydsl.couchbase.document.CouchbaseDocumentSerializer;
27+
import org.springframework.data.couchbase.querydsl.document.AbstractCouchbaseQueryDSL;
28+
import org.springframework.data.couchbase.querydsl.document.CouchbaseDocumentSerializer;
2929

3030
/**
3131
* @author Michael Reiche

src/main/java/org/springframework/data/couchbase/repository/support/SpringDataCouchbaseSerializer.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import java.util.Set;
2222
import java.util.regex.Pattern;
2323

24-
import com.querydsl.couchbase.document.CouchbaseDocumentSerializer;
24+
import org.springframework.data.couchbase.querydsl.document.CouchbaseDocumentSerializer;
2525
import org.springframework.data.couchbase.core.convert.CouchbaseConverter;
2626
import org.springframework.data.couchbase.core.mapping.CouchbasePersistentEntity;
2727
import org.springframework.data.couchbase.core.mapping.CouchbasePersistentProperty;

0 commit comments

Comments
 (0)