persvr / rql

Resource Query Language
http://www.persvr.org/
267 stars 49 forks source link

aggregate returns different field names #62

Open wshager opened 9 years ago

wshager commented 9 years ago

See #61 The test fails because the field is renamed to '0'. This part of the implementation seems to be undocumented.

wshager commented 9 years ago

I updated the test to reflect the undocumented feature. I'll leave this open because of the omission in the specification.

wshager commented 9 years ago

See #63

neonstalwart commented 9 years ago

@wshager it seems that the '0' corresponds to the index of the aggregation function. this makes it possible to do multiple aggregations on the same data and extract each result independently.

wshager commented 9 years ago

I assume Kris will have to update the specs? Hopefully he can find a minute to do that.

neonstalwart commented 9 years ago

yeah, you're probably not happy to hear this, but personally i'm not interested in the specs so they won't get any of my time - the code is all the spec i have need for right now. my main concern is keeping this codebase stable. my limited energy will only be invested towards that for now.

wshager commented 9 years ago

@neonstalwart thanks no problem