Sh-Zh-7 opened a new pull request, #11932:
URL: https://github.com/apache/iotdb/pull/11932

   ## Description
   IoTDB uses `TypeProvider` class to **identify type by given symbol**. And 
aggregators have two output types, one for partial output, another for final 
output.
   
   In current variance and standard deviation aggregators, **the two types are 
different**. The partial output type is `TEXT` while the final output type is 
depend on user's input, which can be `INT`, `FLOAT` and so on.
   
   However, current implementation outputs **same symbols** for both partial 
and final aggregators for variance and standard deviation. And this may cause 
some unexpected behavior like the final output type can be overrided by partial 
output type. 
   
   This PR adds `_partial` suffix to partial output symbol in order to 
distinguish from final output symbol. Notice **there is no need to add new 
integration tests** for this scenario, since #11581 already add `GROUP BY 
LEVEL` in specific test file, which  
   has covered this problem.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [x] been tested in a test IoTDB cluster.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to