avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philip Zeyliger (JIRA)" <j...@apache.org>
Subject [jira] Commented: (AVRO-199) Clean up schemas in testio.py
Date Mon, 16 Nov 2009 05:46:39 GMT

    [ https://issues.apache.org/jira/browse/AVRO-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12778218#action_12778218
] 

Philip Zeyliger commented on AVRO-199:
--------------------------------------

Jeff, I haven't looked at your patch yet, but you might want to look at http://docs.python.org/library/textwrap.html#textwrap.dedent
.  I bet it does something similar to "remove_whitespace", and it's in the stdlib since 2.3.

-- Philip

> Clean up schemas in testio.py
> -----------------------------
>
>                 Key: AVRO-199
>                 URL: https://issues.apache.org/jira/browse/AVRO-199
>             Project: Avro
>          Issue Type: Improvement
>          Components: python
>            Reporter: Jeff Hammerbacher
>         Attachments: AVRO-199.patch, AVRO-199.patch
>
>
> As discussed on the mailing list, it would be nice to leverage Python's expressive string
syntax to not perform lots of escaping of Avro schemas defined in code and make the schemas
more readable.
> This patch cleans up schemas in testio.py to make them more readable. Note that to get
this to pass the tests, I had to implement a new "remove_whitespace()" helper function to
replace the current naive approach to replacing whitespace (s.replace(" ", "") is not going
to cut it). I also object to the round-trip string conversion test, as noted in the comments
to this patch, but I'll save that change for another patch to better separate concerns.
> If this code change is deemed useful, I'll convert the rest of the schemas in the Python
code to leverage Python's string formatting. I have a ridiculously naive tool that I used
for schema conversion that I threw up at http://github.com/hammer/avro-tools/blob/master/pprint_avsc
for those interested.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message