Ignore data in an entity if not matched by fields in the schema, rather than crashing.

Open shans opened this issue 5 years ago • 3 comments

Sep 09 '20 07:09 shans

Note: DON'T LAND THIS PATCH JUST YET! I want to make sure that #6104 resolves b/167648998 first and this will mask the crash.

Sep 09 '20 07:09 shans

Can this be merged now?

Sep 22 '20 22:09 galganif

I'm really unsure about whether we actually want this or not. It does prevent us from crashing if the schema doesn't match the data, but on the flipside it means that we don't necessarily see errors due to this until they're further uo the stack & harder to debug.

Sep 28 '20 01:09 shans