You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2021/11/23 03:45:08 UTC

[GitHub] [iotdb] liuminghui233 opened a new pull request #4451: Use statistics of time column in canUseCurrent*Statistics() method in AlignedSeriesAggregateReader

liuminghui233 opened a new pull request #4451:
URL: https://github.com/apache/iotdb/pull/4451


   In aggregation, we need to judge whether we can use the current file/chunk/page statistics for calculations. For aligned timeseries, we should use the statistics of the time column for judgment.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [iotdb] coveralls edited a comment on pull request #4451: Use statistics of time column in canUseCurrent*Statistics() method in AlignedSeriesAggregateReader

Posted by GitBox <gi...@apache.org>.
coveralls edited a comment on pull request #4451:
URL: https://github.com/apache/iotdb/pull/4451#issuecomment-976161374


   
   [![Coverage Status](https://coveralls.io/builds/44473370/badge)](https://coveralls.io/builds/44473370)
   
   Coverage increased (+0.01%) to 67.278% when pulling **2aafbcda293e29fc656aaf1800bcbafa0878445a on liuminghui233:vector_canUseCurrentStatistics_debug** into **c4d73e89cc413e73be5fac1f5325ba2515c51860 on apache:master**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [iotdb] coveralls edited a comment on pull request #4451: Use statistics of time column in canUseCurrent*Statistics() method in AlignedSeriesAggregateReader

Posted by GitBox <gi...@apache.org>.
coveralls edited a comment on pull request #4451:
URL: https://github.com/apache/iotdb/pull/4451#issuecomment-976161374


   
   [![Coverage Status](https://coveralls.io/builds/44471369/badge)](https://coveralls.io/builds/44471369)
   
   Coverage increased (+0.002%) to 67.06% when pulling **29c09a02494802bf709d10fe67b183b0e8c4711c on liuminghui233:vector_canUseCurrentStatistics_debug** into **c6744c53f5cb16dec9ef74a419ce6028a60e96a0 on apache:master**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [iotdb] coveralls commented on pull request #4451: Use statistics of time column in canUseCurrent*Statistics() method in AlignedSeriesAggregateReader

Posted by GitBox <gi...@apache.org>.
coveralls commented on pull request #4451:
URL: https://github.com/apache/iotdb/pull/4451#issuecomment-976161374


   
   [![Coverage Status](https://coveralls.io/builds/44471244/badge)](https://coveralls.io/builds/44471244)
   
   Coverage decreased (-0.009%) to 67.049% when pulling **29c09a02494802bf709d10fe67b183b0e8c4711c on liuminghui233:vector_canUseCurrentStatistics_debug** into **c6744c53f5cb16dec9ef74a419ce6028a60e96a0 on apache:master**.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [iotdb] JackieTien97 merged pull request #4451: Use statistics of time column in canUseCurrent*Statistics() method in AlignedSeriesAggregateReader

Posted by GitBox <gi...@apache.org>.
JackieTien97 merged pull request #4451:
URL: https://github.com/apache/iotdb/pull/4451


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org