Do we have a test for the case where the cache is
queried while being populated? Remainder queries
should include partially populated regions in this
case. The test may be there somewhere but I don't see
it.
actually that's a very good point, we don't have a testcase
to cover such scenario.
Your comment is very accurate, we must include the partially
populated regions in the remainder query. We could achieve
this by executing the QueryResultStore insertions and the
ChunkStore updates in one transaction, so they won't be seen
by other threads until after the transaction is commited.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=101901
actually that's a very good point, we don't have a testcase
to cover such scenario.
Your comment is very accurate, we must include the partially
populated regions in the remainder query. We could achieve
this by executing the QueryResultStore insertions and the
ChunkStore updates in one transaction, so they won't be seen
by other threads until after the transaction is commited.