Skip to content

Removing datastore key query filter restriction. - #918

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:fix-917
Jun 10, 2015
Merged

Removing datastore key query filter restriction.#918
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:fix-917

Conversation

@dhermes

Copy link
Copy Markdown
Contributor

Also checking in Query constructor that all the filters are valid.

Fixes#917.


@pcostell Can you verify that key inequality filters work? (The docs clearly state they do, just want to be careful.) This has been in the codebase longer than I've been working on the project, so maybe it was introduced before inequality filters were allowed?

Also checking in `Query` constructor that all the filters
are valid.
Fixesgoogleapis#917.
@dhermesdhermes added the api: datastore Issues related to the Datastore API. label Jun 8, 2015
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 8, 2015
@pcostell

Copy link
Copy Markdown
Contributor

Yes, key inequalities are allowed.

@tseaver

Copy link
Copy Markdown
Contributor

@pcostell does the API allow arbitrary rich comparisons on keys (<, <=, >, >=), or just "not equal" (!=). If ordering comparisons are allowed, how is the ordering done?

This comment was marked as spam.

@dhermes

Copy link
Copy Markdown
ContributorAuthor

@tseaver check out #917 for your answer

@tseaver

Copy link
Copy Markdown
Contributor

I see that @vinays uses >= as the operator in his workaround: I'm just not sure whether / how the back-end interprets ordering comparisons on keys (as opposed to simple equality / inequality).

@pcostell

Copy link
Copy Markdown
Contributor

Yes I mean <, >, <=, and >=. Like property filters, != is not supported. Ordering is performed on the key path tuples and names are always greater than ids. So
(MyParent, 123), (MyKind, "bar") > (MyParent, 123), (MyKind, 12) > (MyParent, 123), (MyKind, 6) > (MyParent, 123) > (MyParent, 122)

@tseaver

Copy link
Copy Markdown
Contributor

@pcostell OK, thanks. For future reference, I found the docs for key filters which describe those semantics (but not as clearly as you did :).

@tseaver

Copy link
Copy Markdown
Contributor

@dhermes LGTM

dhermes added a commit that referenced this pull request Jun 10, 2015
Removing datastore key query filter restriction.
@dhermes
dhermes merged commit 319963f into googleapis:masterJun 10, 2015
@dhermes
dhermes deleted the fix-917 branch June 10, 2015 19:00
@dhermesdhermes mentioned this pull request Jul 10, 2015
parthea pushed a commit that referenced this pull request Mar 2, 2026
* feat: add cloud storage subscription fields
PiperOrigin-RevId: 531202368
Source-Link: googleapis/googleapis@8a4cc94
Source-Link: googleapis/googleapis-gen@ec60ad7
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZWM2MGFkNzYzNTZkODhlOTlmYTQ4NDRmYmQ5MGZkY2NhNzI4ZjVjYiJ9
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Mar 9, 2026
…ads, as a preview feature (#943)
* checkpoint before design doc impl
* checkpoint
* more tests
* code and tests for transfer manager complete
* proactively close temp files when finished reading
* respond to comments; destroy tmp files as they are consumed
* Add system tests, docstrings, address feedback
* Respond to review comments
* verify md5 hash of downloaded file in test
* lint
* default empty strings for root arguments
* fix bug with blob constructor
* add warning about files not being deleted if their downloads fail
* docs: Add samples to multithread branch (#918)
* add samples, tests pending
* add snippet tests
* snippet and snippets_test.py linting
* snippets; recursive directory creation; rename some params
* Add directory upload snippet
* fix: remove chunked downloads; change max_workers to threads
* update snippets to add thread info
* fix snippets test issue due to change in dependency
* snippet nomenclature
* fix samples for real this time
parthea pushed a commit that referenced this pull request Mar 9, 2026
* docs: Fix formatting of request arg in docstring
chore: Update gapic-generator-python to v1.9.1
PiperOrigin-RevId: 518604533
Source-Link: googleapis/googleapis@8a085ae
Source-Link: googleapis/googleapis-gen@b2ab4b0
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYjJhYjRiMGEwYWUyOTA3ZTgxMmMyMDkxOThhNzRlMDg5OGFmY2IwNCJ9
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: datastoreIssues related to the Datastore API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@dhermes@pcostell@tseaver@googlebot