[GEODE-9449] remove 'b' prefix from constants

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.15.0
• Fix Version/s: 1.15.0
• Component/s: redis
• Labels: pull-request-available
• Assignee: Kristen
• Reporter: Darrel Schneider
• Created: 23/Jul/21 17:09
• Updated: 22/Jun/22 20:45
• Resolved: 18/Nov/21 00:50

Description

I number of constants in the redis packages have a 'b' prefix on their name. This might have been a use of Hungarian notation but that is not clear. The convention for constant names in geode is all upper case with underscore between words. So the 'b' prefix should be removed. See StringBytesGlossary for the location of many of these constants. Most of the constants in StringBytesGlossary contains bytes, one or more, but if a few of the constants in it are actually String instances. Consider renaming them to have _STRING suffix or moving them to another class like StringGlossary. 

The byte array constants in this class are marked with the MakeImmutable annotation. 

Issue Links

• links to: GitHub Pull Request #7118