Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upEvaluation rules order of evaluation undefined #1088
Comments
This comment has been minimized.
This comment has been minimized.
|
The agreed solution is to be able to have groups of rules, and that they will be ordered within the group. This still allows use of multiple cores for calculations. I suspect that #17 will never happen. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
On Thu, Sep 17, 2015 at 6:42 PM Björn Rabenstein notifications@github.com
|
This comment has been minimized.
This comment has been minimized.
|
|
juliusv
referenced this issue
Sep 17, 2015
Closed
Support grouping rules such that rules within a group are executed sequentially #1095
This comment has been minimized.
This comment has been minimized.
|
Superseded by #1095. Closing. |
juliusv
closed this
Sep 17, 2015
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 24, 2019
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
aecolley commentedSep 16, 2015
It is impossible to use a recorded value in another recorded rule, because the evaluation order is unpredictable. Sometimes, you get the value from the current evaluation, and sometimes you get the result of the previous evaluation. This leads to oddities like dividing this minute's sum1m by last minute's count1m, giving wrong values and the occasional +Inf.
Suggested fix: add a command-line flag to enforce sequential evaluation of rules in the same order as their appearance in the rules file.
This is related to issue 17, but that is about adopting some sophisticated automatic sorting, whereas this bug is about the lack of any basic consistency. I would be unhappy if this wasn't going to be fixed until issue 17 was ready.