GitHub user BruceKuiLiu opened a pull request:
https://github.com/apache/activemq-artemis/pull/2037
ARTEMIS-1822: Change the method name 'lookupRecord' to 'containsRecord'.
The 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".
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/BruceKuiLiu/activemq-artemis master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/2037.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2037
----
commit 2b3bdf4f5b7e1191481958324f34783b0b983047
Author: Kui LIU <brucekuiliu@...>
Date: 2018-04-21T21:46:47Z
Change the method name 'lookupRecord' to 'containsRecord'.
----
---
