accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki (JIRA)" <j...@apache.org>
Subject [jira] [Created] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public
Date Tue, 03 Dec 2013 17:46:36 GMT
Bill Havanki created ACCUMULO-1958:
--------------------------------------

             Summary: 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.6.0, 1.7.0
            Reporter: Bill Havanki


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#6144)

Mime
View raw message