[GitHub] [druid] clintropolis opened a new pull request #11198: clarify that appenderator persist byte sizes are estimations

[GitBox <gi...@apache.org>]

clintropolis opened a new pull request #11198:
URL: https://github.com/apache/druid/pull/11198


   These log messages that appear whenever an indexing task persists segments:
   ```
   2021-05-04T03:03:50,004 INFO [task-runner-0-priority-0] org.apache.druid.segment.realtime.appenderator.AppenderatorImpl - Persisted rows[3,519] and bytes[477,231,910]
   ```
   are printing the estimated bytes in memory, but it isn't clear that this is an estimated value, not the actual number of bytes written, which is confusing. This PR adjusts the logging to clarify that these values are not actual byte sizes (which might be nice to have available so we can compare how far our estimates are off from reality, but I'll save for another day).


-- 
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

[GitHub] [druid] asdf2014 merged pull request #11198: clarify that appenderator persist byte sizes are estimations

[GitBox <gi...@apache.org>]

asdf2014 merged pull request #11198:
URL: https://github.com/apache/druid/pull/11198


   


-- 
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org