[ARTEMIS-1822] Inconsistent method name

• Type: Task
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 2.6.0
• Component/s: Broker
• Assignee: Unassigned
• Reporter: KuiLIU
• Created: 21/Apr/18 21:51
• Updated: 17/May/18 02:43
• Resolved: 17/May/18 02:43

Description

The following method is named "lookupRecord".
"lookupRecord" seems to find a related record.
But the method is checking whether recordsSnapshot contains the id or not.
Thus, the method name "containsRecord" is more intuitive than "lookupRecord".

   public boolean lookupRecord(final long id) {
      return recordsSnapshot.contains(id);
   }

Issue Links

• links to: GitHub Pull Request #2037