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
review feedback
  • Loading branch information
@kolea2
kolea2 committed Jan 18, 2024
commit f23e1ecdf732bd543b683ae915b6a582ed58c275
2 changes: 1 addition & 1 deletion google-cloud-datastore/src/main/java/com/google/cloud/datastore/Datastore.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -485,7 +485,7 @@ interface TransactionCallable<T> {
@BetaApi
default <T> QueryResults<T> run(
Query<T> query, QueryProfile.QueryMode queryMode, ReadOption... options) {
throw new UnsupportedOperationException("not implemented");
throw new UnsupportedOperationException("Not implemented.");
}

/**
Expand Down
3 changes: 2 additions & 1 deletion google-cloud-datastore/src/main/java/com/google/cloud/datastore/QueryResultsImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,8 @@ private void sendRequest() {
boolean isQueryPlan = actualResultType.getQueryType() == null && queryMode == QueryMode.PLAN;
Preconditions.checkState(
queryResultType.isAssignableFrom(actualResultType) || isQueryPlan,
"Unexpected result type " + actualResultType + " vs " + queryResultType);
"Unexpected result type or query mode. Result type: " + actualResultType + " vs " + queryResultType +", query mode: " + queryMode);

if (runQueryResponsePb.getStats().hasQueryPlan()
|| runQueryResponsePb.getStats().hasQueryStats()) {
this.resultSetStats = new ResultSetStats(runQueryResponsePb.getStats());
Expand Down