Code review comment for lp://staging/~free.ekanayaka/storm/schema-uri-in-testresource

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

As a minor nit, it feels like it's time to introduce a richer object to hold that information. There's no reason to carry around that kind of untyped blob and have to parse it out when reading.

Note that *everything* can be represented as a key and a value in a dict in a list in a ... still, we use types and instances with nicely named methods, etc.

« Back to merge proposal