Reading from backup does not update access data

[GhostInAMachine]

When MapConfig option "readBackupData" is set to true then "get" operations do not trigger the update of access timestamps and hit count. This way the entries might get evicted even though they are frequently used.

Cause of this issue is in com.hazelcast.map.impl.DefaultRecordStore#readBackupData method.

[gurbuzali]

Let's keep in mind that even if we update access-time/hit-count on DefaultRecordStore.readBackupData it will be on backup partition, owner partition will not be affected. Therefore the owner will evict the entry but the backup will retain.

[gurbuzali]

Another point: We do not update localMapStats.getCount too (which are kept locally and updated in invokeOperation method of proxy ).

[tombujok]

@gurbuzali @jerrinot @ahmetmircik The problem here is that when readBackupData == true when you do a get for a key that's on a partition with a replicaIndex == 0 on this node, the partitionThread will be bypassed and the stats will not be updated. That's a bug IMHO. This fix submitted (but not merged) by @jerrinot was correct: #8012 Any objections to merge it?

[tombujok]

I'm moving it to 3.8 since we need the caller runs feature to implement it.
It's unsolvable in 3.6 or 3.7 without performance regression.
Some more comments here: #8489