[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-12 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846418#comment-13846418 ] ASF subversion and git services commented on ACCUMULO-1958: ---

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-12 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846420#comment-13846420 ] ASF subversion and git services commented on ACCUMULO-1958: ---

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-12 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846410#comment-13846410 ] ASF subversion and git services commented on ACCUMULO-1958: ---

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-12 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846413#comment-13846413 ] ASF subversion and git services commented on ACCUMULO-1958: ---

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-12 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846411#comment-13846411 ] ASF subversion and git services commented on ACCUMULO-1958: ---

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-12 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13846409#comment-13846409 ] ASF subversion and git services commented on ACCUMULO-1958: ---

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-09 Thread Bill Havanki (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13843282#comment-13843282 ] Bill Havanki commented on ACCUMULO-1958: Review updated: the Thrift constructor

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-06 Thread Bill Havanki (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13841377#comment-13841377 ] Bill Havanki commented on ACCUMULO-1958: Review available. > Range constructor

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840435#comment-13840435 ] Keith Turner commented on ACCUMULO-1958: bq. For 1.4.x and 1.5.x, though, I thi

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Bill Havanki (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840432#comment-13840432 ] Bill Havanki commented on ACCUMULO-1958: Works for me. The only users of the no

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Sean Busbey (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840422#comment-13840422 ] Sean Busbey commented on ACCUMULO-1958: --- I presume the non-range-checking version

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Bill Havanki (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840413#comment-13840413 ] Bill Havanki commented on ACCUMULO-1958: I'd be OK with leaving out the range c

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840399#comment-13840399 ] Keith Turner commented on ACCUMULO-1958: Another way to look at it is that its

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Bill Havanki (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840398#comment-13840398 ] Bill Havanki commented on ACCUMULO-1958: OK. That implies then, that while the

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840391#comment-13840391 ] Keith Turner commented on ACCUMULO-1958: Just checked 1.4 and 1.5, The method e

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840386#comment-13840386 ] Keith Turner commented on ACCUMULO-1958: Since its public it would first need t

[jira] [Commented] (ACCUMULO-1958) Range constructor lacks key checks, should be non-public

2013-12-05 Thread Bill Havanki (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13840362#comment-13840362 ] Bill Havanki commented on ACCUMULO-1958: My preferred solution for this is to m