Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/3721
Okay, taking a step back. Looking through the code some more, the internal
arithmetric should certainly stay in bytes. However, bytes are tedious to
configure.
I suggest to add support to the configuration to interpret memory units, so
that we can configure values via
- 512m
- 10 kb
- ...
I have started some utility here:
https://github.com/StephanEwen/incubator-flink/tree/mem_size
That means that we would keep the PR in this form and add memory
configuration parsing as a followup.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
