Created
October 11, 2010 22:39
-
-
Save PharkMillups/621355 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
12:18 <technomancy> hello, fine gentlemen | |
12:19 <technomancy> got a question about mapreduce jobs and the format in | |
which their arguments are provided | |
12:19 <technomancy> I'd expect if you PUT with a content-type of application/json, | |
the data would be passed into the map function as a javascript object rather than a string | |
12:20 <technomancy> that seems to not be the case: http://p.draines.com/12865618750730d9a9e6c.txt | |
12:20 <technomancy> that returns [{"class":"String","data":"\"{\\\"bar\\\":\\\"baz\\\"}\""}], for the curious | |
12:20 <seancribbs> technomancy: Riak doesn't auto-parse the contents | |
for you fit that's what you mean | |
12:20 <seancribbs> s/fit/if/ | |
12:21 <technomancy> so why does mapValuesJSON bother to check if the class is an array | |
if it's always going to be a string? | |
12:23 <seancribbs> technomancy: i have no idea why it would be an array at all… | |
that seems broken | |
12:23 <seancribbs> or at least, naive | |
12:23 <seancribbs> it should be a string at all times | |
12:23 <seancribbs> that is, each value | |
12:25 <technomancy> ok, so it's just up to the jobs to do the parsing; gotcha | |
12:26 <seancribbs> yes | |
12:26 <technomancy> thanks |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment