AND operator precedence is too high

[aecolley]

The expression:
time() - start_time{job="x"} > 86400
    and
start_time{job="x"} > end_time{job="x"}

should be parsed as:
(time() - start_time{job="x"} > 86400)
    and
(start_time{job="x"} > end_time{job="x"})

but it is actually parsed as:
((time() - start_time{job="x"} > 86400)
    and
start_time{job="x"}) > end_time{job="x"}

rules/lexer.l classifies AND and OR as CMP_OP just like == and >. They should be a different level of operation.

[juliusv]

Thanks for reporting! I agree, this should be changed.

[atombender]

+1. This bit me today. At the very least the current precedence should be documented.

[juliusv]

This will be fixed along with the parser / query engine refactoring that is likely to be done in the next couple of weeks.

[juliusv]

Just merged the new query engine + parser feature branch into master: #676

This should be fixed now.

[lock]

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.