Agree, looks inconsistent to me. Alex G., could you chime in?
On Mon, Apr 24, 2017 at 6:30 PM, Pavel Tupitsyn <ptupit...@apache.org> wrote: > Igniters, > > I have noticed quite a lot of inconsistencies with the rest of our APIs in > our new MemoryMetrics API: > > 1) MemoryMetrics is not a snapshot. It is a "live" instance which returns > different values each time you access some property. (IgniteCache.metrics() > returns a snapshot). > > 2) MemoryMetrics has methods that modify state, like enableMetrics > and rateTimeInterval > (IgniteCache.metrics() returns a read-only serializable snapshot) > > 3) getSize method returns size in megabytes > (IgniteCache.metrics() provides sizes in bytes) > > > I propose to rework this API until it is not too late. > > Thoughts? >
