@tsteur opened this Pull Request on September 5th 2017 Owner

fixes #12003
Did a simple test locally. Without this patch I see no data, with patch I see existing data from eg a week archive.

Not sure if this would have any downsides but can't think of any.

@tsteur commented on September 5th 2017 Owner

Actually, there is an index on period which is why the period = ? is there I think but the index already matches the first three columns of the index

(`idsite`,`date1`,`date2`,`period`,`ts_archived`)

so it should be fine I think and still fast

@mattab commented on September 18th 2017 Owner

Also can't think of a downside, considering the INDEX would still be used.

Nice idea :+1:

This Pull Request was closed on September 18th 2017
Powered by GitHub Issue Mirror