truncate positive infinity to today, don't output if negative infinity is part of a count #2820
Conversation
…y is part of a count
| public DurationSumAggregator(Column column) { | ||
| super(column); | ||
| } | ||
|
|
||
| @Override | ||
| public void init(Entity entity, QueryExecutionContext context) { | ||
| set.clear(); | ||
| realUpperBound = context.getToday(); |
There was a problem hiding this comment.
Ich denke nicht, dass das das ist was wir wollen, so kann eine Anfrage von gestern schon ganz andere Ergebnisse haben wenn sie Heute oder demnächst wiederverwendet wird.
Um das zu verhindern, sollte die upperBound mit persistiert werden, bzw. eine DateRestiction gesetzt sein. Sonst sehe ich Probleme bei der Reproduzierbarkeit von Ergebnissen
There was a problem hiding this comment.
Das machen wir bei DateDistance genauso, und ist auch genau das Ziel des PRs
There was a problem hiding this comment.
Jap, ich weiß, aber das könnte Probleme mit sich bringen
| 5,"2011-03-05/2011-03-13" | ||
| 6,"/2011-03-13" |
There was a problem hiding this comment.
Kein Case für "2011-03-13/" ?
There was a problem hiding this comment.
würde sich ja ändern mit dem heutigen Datum
There was a problem hiding this comment.
das würde sich auch bei der erstausführung testen, was es ja in den tests ist.
No description provided.