reddmetrics/forma-clj — pull request #67 by robinkraft (2012-07-09 22:27:15+00:00)
Title: Feature/fix one off

This pull request restores a `dec` to a function that calculates the ending period of a timeseries based on the length of the timeseries array. The end period generated here without `dec` technically is incorrect. Fortunately, after reading through the subsequent usage of this function, I've determined that the one-off error does not affect the quality of output data, since it is only ever used for [ordering values](https://github.com/reddmetrics/forma-clj/blob/feature/deliver/src/clj/forma/hadoop/jobs/forma.clj#L153) - relative order, rather than absolute order based on dates or periods.

`?end` appeared [here](https://github.com/reddmetrics/forma-clj/blob/feature/deliver/src/clj/forma/hadoop/jobs/forma.clj#L201) but `p/index` just below it is actually used to ensure the correct ordering in that query. As a result, I've replaced `?end` with `_`