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


With regards,
Apache Git Services

Reply via email to