-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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.
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
[#10955] Wrap up Objectify V6 migration #11058
Conversation
9f51def
to
b5ce17b
Compare
* update objectify dependency to v6 and fix initial build issues * reset localhost port to default * updated cloud datastore cursor api for v6 * add missing Objectify.init() in base test case * add project id information into Objectify.init() * fixed wrong order from auto import * include localDatastoreHelper test config * fixed deleteEntity key id compatibility to v6 * updated e2e test setup for local datastore helper * remove local datastore service test config * remove not equal DB operation * added local datastore helper for e2e testing * Generate legacy key format consistent with the old appengine SDK * moved localDatastoreHelper setup from OfyHelper to base test case * moved datastore port config to build.template.properties * improved on documentation * refactored emulator port config Co-authored-by: jianhandev <jianhanli.dev@gmail.com>
* Replace usage of Text type from appengine datastore api with StringValue * Use unindexed String as alternative over StringValue previously used to replace Text * Simplify getters and setters * Remove redundant architecture test method after deprecating use of datastore Text type
#11025) * Remove usage of DatastoreTimeoutException from old api * Get WebApiServlet to throw specific datastore error messages * Replace hard-coded values with enum values of google.rpc.Code
95a2b4e
to
f768ad3
Compare
@Derek-Hardy @jianhandev please help to do one last sanity check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, should we include the documentations (for datastore emulator) as well before merging it into teammatesv8
branch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Maybe sync up this documentation as well.
Fixes #10955
This PR will symbolically move all the commits from the Objectify v6 migration PRs to the longer-living
teammatesv8
branch.Merging shall be using the rebase-and-merge option.