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

RFE: automatically use datastore Text for properties greater than 500 characters #31

Closed
mblakele opened this issue Jul 18, 2011 · 6 comments

Comments

@mblakele
Copy link

Google's datastore limits strings to 500 characters. It would be nice if the datastore service handled this magically.

(if (< (.length src) 500) value (com.google.appengine.api.datastore.Text. value)
@gcv
Copy link
Owner

gcv commented Jul 18, 2011

I'll try to do this for the next version. Patches always welcome, naturally. :)

@jurrchen
Copy link
Contributor

I think this is something that should be handled in your application code instead of appengine-magic. TextProperties are very different from StringProperties (for example, TextProperties cannot be indexed) and alot of things could go wrong if we start automatically converting from strings to text instead of failing loudly and obnoxiously when the user tries to store a string > 500 characters.

@gcv
Copy link
Owner

gcv commented Aug 13, 2011

Good point about different behavior. I'll think about this some more before coding it up.

@gcv
Copy link
Owner

gcv commented Sep 22, 2011

I'm leaning towards not supporting this automatic type change. Seems like it has too much potential for introducing subtle bugs.

@gcv gcv closed this as completed Sep 22, 2011
@mblakele
Copy link
Author

A doc example would be about as useful as a new feature would be.

@gcv
Copy link
Owner

gcv commented Sep 22, 2011

Not sure what you're talking about..?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants