alltyme / appengine-ndb-experiment

Automatically exported from code.google.com/p/appengine-ndb-experiment
Other
0 stars 0 forks source link

Properties can't be added to structured properties #220

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Upload an app with a local structured property X, with property Y, and 
version 1.
2. Upload version 2 of an app adding property Z to X.  Do a request to version 
2.
3. Version 1 now fails reading the record put by version 2 with
RuntimeError('Unable to find property %s of StructuredProperty %s.'
                         % (next, self._name))

What is the expected output? What do you see instead?
  The unknown property should be ignored.

What version of the product are you using? On what operating system?
__version__ = '1.0.6' included with the appengine 1.7.3

Please provide any additional information below.

Original issue reported on code.google.com by jim.morr...@gmail.com on 2 Nov 2012 at 8:03

GoogleCodeExporter commented 9 years ago
Can you show some more concrete code that I could run to reproduce your 
problem? 

Original comment by guido@google.com on 2 Nov 2012 at 8:40

GoogleCodeExporter commented 9 years ago
NM just saw your patch. :-)

Original comment by guido@google.com on 2 Nov 2012 at 8:40

GoogleCodeExporter commented 9 years ago
Does the test in issue 6819079 help?

On Fri, Nov 2, 2012 at 1:40 PM,
<appengine-ndb-experiment@googlecode.com> wrote:

Original comment by jim.morr...@gmail.com on 2 Nov 2012 at 8:41

GoogleCodeExporter commented 9 years ago
This issue was closed by revision df5283f902ed.

Original comment by guido@google.com on 19 Nov 2012 at 5:46

GoogleCodeExporter commented 9 years ago
Code review of the change committed was: http://codereview.appspot.com/6854065/

Original comment by gvanrossum@gmail.com on 19 Nov 2012 at 5:57

GoogleCodeExporter commented 9 years ago
Also, I posted some background to the code modified here: 
https://groups.google.com/d/msg/appengine-ndb-discuss/1E8pMkpAB6A/W401a-nU03gJ

Original comment by gvanrossum@gmail.com on 19 Nov 2012 at 6:00

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 63ff0731b3ab.

Original comment by guido@google.com on 19 Nov 2012 at 9:38