[GitHub] zentol commented on issue #6962: change accumulator value to be formatted with digit group separator

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

zentol commented on issue #6962: change accumulator value to be formatted with digit group separator
URL: https://github.com/apache/flink/pull/6962#issuecomment-461523734
 
 
   I would like to understand what case we're trying to solve here. What locality-dependent component does a stringified number have? The only case that comes to my mind right now are periods vs. commas, but this wouldn't be addressed here.
   
   Beyond that, accumulators don't impose an restriction on the value type, so albeit being admittedly a rare case we can't merge this.
   
   FYI, the `number` filter will return en empty string for non-numeric values.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services