Repository navigation
datastore: default to empty dataset #121
Description
When working with datasets, the Datastore backend will normalize any empty datasets to the dataset of the calling application. This should be preferred to prepending "s~" to dataset ids as that is not always a valid operation.
Activity
tseaver commentedon Sep 22, 2014
The 'to_protobuf' method of gcloud.datastore.key.Key jams a 's~' prefix onto the key's dataset_id, if the ID does not already have a 's~' or 'e~' prefix:
https://github.com/GoogleCloudPlatform/gcloud-python/blob/1f2aaf3606802f4f02eeb30e22e1082ef7f4594e/gcloud/datastore/key.py#L60
@pcostell Is that the normalization you are thinking of?
pcostell commentedon Sep 22, 2014
Yes that's the normalization that is not needed. The datastore backend is smart about looking up the correct dataset for the application.
Merge pull request #259 from tseaver/121-drop_dataset_prefixes_in_keys
tseaver commentedon Oct 20, 2014
The fix in #259 is not correct / complete. See:
Revert "Fix #121: Drop munging of 's~' prefix onto keys."
Merge pull request #274 from GoogleCloudPlatform/revert-259-121-drop_…
dhermes commentedon Oct 21, 2014
You'd figure GitHub wouldn't auto-close on their own reverts?
99 remaining items
chore: update dependency protobuf >= 3.20.2 (#121)
[feat] Add automatic pagination support. (#121)
chore: release 1.9.1 (#121)
chore: release 0.6.1 (#121)
chore: fix kokoro config for samples (#121)
feat: add support for Python 3.10 (#121)
feat: asyncio microgen document (#121)
initial version of migration guide (#121)
docs: Add a link to Pub/Sub filtering language public documentation t…
fix: add empty slots to CommonChecksum (#121)
test: drop majyk per-lang-version coverage level (#121)
django, dbapi: add adapt_decimalfield_value for Decimal->float
feat: add helper for bucket bound hostname URLs (#137)
chore: move Renovate config to .github, turn off dashboard (#121)
fix: add empty slots to CommonChecksum (#121)