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.

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

Allow ExecutionContext.newObjectId(Class, Object) to pass in key for datastore id #366

Closed
andyjefferson opened this issue Mar 12, 2021 · 0 comments
Closed

Allow ExecutionContext.newObjectId(Class, Object) to pass in key for datastore id #366

andyjefferson opened this issue Mar 12, 2021 · 0 comments
Labels
enhancement
Milestone
5.2.7

Comments

@andyjefferson
Copy link
Member

The JDO spec only requires the second parameter to be a toString() value. We could allow the key (when numeric for example) to be passed in when using datastore id.
@andyjefferson andyjefferson added this to the 5.2.7 milestone Mar 12, 2021
@andyjefferson andyjefferson changed the title Allow newObjectId(Class, Object) to pass in key for datastore id Allow ExecutionContext.newObjectId(Class, Object) to pass in key for datastore id Mar 12, 2021
andyjefferson added a commit that referenced this issue Mar 12, 2021
@andyjefferson
Allow datastore id with input of a numeric being the key object. Fixes
e3414d1 
#366
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
5.2.7
Development

No branches or pull requests
1 participant
@andyjefferson