----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9376/#review16370 -----------------------------------------------------------
Ship it! Two questions to think about in terms of becoming a utility: -This is a very specific utility, seems like it may be better to have this read json from stdin, that way people can curl and pipe it in. -This reminds me of _.pluck (from underscore.js), maybe a name like jsonpluck would be more descriptive (were we to pull out the url loading bit). -Being a utility, do we want to remove the .py suffix? Akin to post-reviews. - Ben Mahler On Feb. 8, 2013, 7:01 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9376/ > ----------------------------------------------------------- > > (Updated Feb. 8, 2013, 7:01 a.m.) > > > Review request for mesos and Vinod Kone. > > > Description > ------- > > See summary. > > > Diffs > ----- > > support/jsonurl.py 0d74b710fda507d64e899f76f55d3c698ae9886f > > Diff: https://reviews.apache.org/r/9376/diff/ > > > Testing > ------- > > > Thanks, > > Benjamin Hindman > >
