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.

Sign up for GitHub

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

Jump to bottom

feat: implement query profile #1365

Merged
merged 19 commits into from
Mar 25, 2024
Merged

feat: implement query profile #1365

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9d0a267
feat: query profiling
kolea2 Dec 13, 2023
b09775e
Merge branch 'main' of github.com:googleapis/java-datastore into qp
kolea2 Jan 4, 2024
c8b5ae3
Merge branch 'main' of github.com:googleapis/java-datastore into qp
kolea2 Jan 18, 2024
ca6a85e
fix clirr, run linter
kolea2 Jan 18, 2024
f23e1ec
review feedback
kolea2 Jan 18, 2024
151ce49
lint again
kolea2 Jan 18, 2024
4a6dc23
switch return type to optional for ResultSetStats
kolea2 Jan 18, 2024
31aa956
Merge branch 'main' of github.com:googleapis/java-datastore into qp
kolea2 Feb 13, 2024
d7bbb36
new api updates
kolea2 Feb 14, 2024
61f2e50
Merge branch 'main' of github.com:googleapis/java-datastore into qp
kolea2 Mar 9, 2024
058bc93
update clirr
kolea2 Mar 10, 2024
08d3563
Merge branch 'main' of github.com:googleapis/java-datastore into qp
kolea2 Mar 13, 2024
a439264
cleanup, add docs
kolea2 Mar 13, 2024
6c041b2
javadoc fixes
kolea2 Mar 13, 2024
12e1278
add betaapi in a few more places
kolea2 Mar 18, 2024
6d6a799
add some extra tests
kolea2 Mar 18, 2024
229fc25
add some more ITs
kolea2 Mar 18, 2024
9f34f34
add some more fine grained tests
kolea2 Mar 19, 2024
2e85328
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
fix clirr, run linter
  • Loading branch information
@kolea2
kolea2 committed Jan 18, 2024
commit ca6a85eb57f96621e4414d56df9e9539136553a7
27 changes: 14 additions & 13 deletions google-cloud-datastore/clirr-ignored-differences.xml
View file
Open in desktop
@@ -1,19 +1,6 @@
<?xml version="1.0" encoding="UTF-8"?>
<!-- see http://mojo.codehaus.org/clirr-maven-plugin/examples/ignored-differences.html -->
<differences>
<!-- renaming internal only class-->
<difference>
<className>com/google/cloud/datastore/ReadOption$QueryAndReadOptions</className>
<method>*</method>
<differenceType>8001</differenceType>
</difference>
<difference>
<className>com/google/cloud/datastore/execution/request/AggregationQueryRequestProtoPreparer</className>
<method>*QueryAndReadOptions*</method>
<to>*QueryConfig*</to>
<differenceType>7005</differenceType>
</difference>

<!-- added a default method to an interface-->
<difference>
<className>com/google/cloud/datastore/Datastore</className>
Expand Down Expand Up @@ -69,4 +56,18 @@
<method>java.lang.Object execute(com.google.cloud.datastore.Query, com.google.cloud.datastore.ReadOption[])</method>
<differenceType>7004</differenceType>
</difference>

<!-- updated internal only class and method-->
<difference>
<className>com/google/cloud/datastore/ReadOption$QueryConfig</className>
<method>*QueryConfig*</method>
<to>*com.google.datastore.v1.QueryMode*</to>
<differenceType>7005</differenceType>
</difference>
<difference>
<className>com/google/cloud/datastore/ReadOption$QueryConfig</className>
<method>com.google.cloud.datastore.ReadOption$QueryConfig create(com.google.cloud.datastore.Query)</method>
<differenceType>7004</differenceType>
</difference>

</differences>
11 changes: 7 additions & 4 deletions google-cloud-datastore/src/main/java/com/google/cloud/datastore/Datastore.java
View file
Open in desktop
Expand Up @@ -465,8 +465,9 @@ interface TransactionCallable<T> {
<T> QueryResults<T> run(Query<T> query, ReadOption... options);

/**
* Submits a {@link Query} with specified {@link com.google.cloud.datastore.models.QueryProfile.QueryMode} and returns its result. {@link ReadOption}s can be specified if
* desired.
* Submits a {@link Query} with specified {@link
* com.google.cloud.datastore.models.QueryProfile.QueryMode} and returns its result. {@link
* ReadOption}s can be specified if desired.
*
* <p>Example of running a query to find all entities of one kind.
*
Expand Down Expand Up @@ -535,8 +536,9 @@ default AggregationResults runAggregation(AggregationQuery query, ReadOption...
}

/**
* Submits a {@link AggregationQuery} with specified {@link com.google.cloud.datastore.models.QueryProfile.QueryMode} and returns {@link AggregationResults}. {@link ReadOption}s
* can be specified if desired.
* Submits a {@link AggregationQuery} with specified {@link
* com.google.cloud.datastore.models.QueryProfile.QueryMode} and returns {@link
* AggregationResults}. {@link ReadOption}s can be specified if desired.
*
* <p>Example of running an {@link AggregationQuery} to find the count of entities of one kind.
*
Expand All @@ -553,6 +555,7 @@ default AggregationResults runAggregation(AggregationQuery query, ReadOption...
* AggregationResults aggregationResults = datastore.runAggregation(aggregationQuery, QueryMode.EXPLAIN_ANALYZE);
* ResultSetStats aggregationStats = aggregationResults.getResultSetStats();
* }</pre>
*
* @throws DatastoreException upon failure
* @return {@link AggregationResults}
*/
Expand Down
3 changes: 2 additions & 1 deletion google-cloud-datastore/src/main/java/com/google/cloud/datastore/DatastoreReader.java
View file
Open in desktop
Expand Up @@ -65,7 +65,8 @@ default AggregationResults runAggregation(AggregationQuery query) {
}

/**
* Submits a {@link AggregationQuery} with a specified {@link com.google.cloud.datastore.models.QueryProfile.QueryMode} and returns {@link
* Submits a {@link AggregationQuery} with a specified {@link
* com.google.cloud.datastore.models.QueryProfile.QueryMode} and returns {@link
* AggregationResults}.
*
* @throws DatastoreException upon failure
Expand Down