[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Bill Havanki updated ACCUMULO-1958: ----------------------------------- Fix Version/s: 1.6.0 1.5.1 1.4.5 Status: Patch Available (was: In Progress) > Range constructor lacks key checks, should be non-public > -------------------------------------------------------- > > Key: ACCUMULO-1958 > URL: https://issues.apache.org/jira/browse/ACCUMULO-1958 > Project: Accumulo > Issue Type: Bug > Affects Versions: 1.4.0 > Reporter: Bill Havanki > Assignee: Bill Havanki > Fix For: 1.4.5, 1.5.1, 1.6.0, 1.7.0 > > Attachments: ACCUMULO-1958.patch > > > This ticket pertains to the Range class constructor with the following > signature: > {noformat} > public Range(Key, Key, boolean, boolean, boolean, boolean) > {noformat} > The constructor does not check that the start key is before the end key, like > every other constructor in the class. Since the constructor is public, this > makes it possible for a caller to create invalid ranges. > The constructor is used by other constructors that take in existing range > objects, where the key check has implicitly been done, and so it would make > sense to skip the check in this one, but then the constructor visibility > should be at least protected. -- This message was sent by Atlassian JIRA (v6.1.4#6159)