Pertaining to this ticket, adds a json_array_agg function to the group_by operator so that users can generate JSON arrays in a single step.
Adds a small test case to earthmover -t that validates this operation. Also fixes a small bug that caused the test code to fail if there was no outputs directory present, and another bug that resulted in some missing data in the animals output.
Additionally, adds information in the README about earthmover init and earthmover clean, which I neglected to do earlier.
Discussion
The way json_array_agg uses _get_agg_lambda's separator argument is questionable. I think its best to leave that argument named separator for the time being instead of thinking of a more generalizable name and/or structure for the _sep object. I've tried to document this quirk, but if reviewers think this is an overload gone too far, I can try to come up with something less ambiguous.
Pertaining to this ticket, adds a
json_array_aggfunction to thegroup_byoperator so that users can generate JSON arrays in a single step.Adds a small test case to
earthmover -tthat validates this operation. Also fixes a small bug that caused the test code to fail if there was nooutputsdirectory present, and another bug that resulted in some missing data in the animals output.Additionally, adds information in the README about
earthmover initandearthmover clean, which I neglected to do earlier.Discussion
The way
json_array_agguses_get_agg_lambda'sseparatorargument is questionable. I think its best to leave that argument namedseparatorfor the time being instead of thinking of a more generalizable name and/or structure for the_sepobject. I've tried to document this quirk, but if reviewers think this is an overload gone too far, I can try to come up with something less ambiguous.