[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread Jesse Yates (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396003#comment-13396003 ] Jesse Yates commented on HBASE-6162: -1 on an hbase-common-tests module until its

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396069#comment-13396069 ] Matt Corgan commented on HBASE-6162: Jesse - have you by chance tried getting tests to

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396128#comment-13396128 ] stack commented on HBASE-6162: -- bq. Jesse - have you by chance tried getting tests to run

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396152#comment-13396152 ] Matt Corgan commented on HBASE-6162: {quote}You don't think it ok just adding new

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396203#comment-13396203 ] Zhihong Ted Yu commented on HBASE-6162: --- I think we should put more efforts into

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread Jesse Yates (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396205#comment-13396205 ] Jesse Yates commented on HBASE-6162: bq. Jesse and I are proposing that we put the

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-18 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13396262#comment-13396262 ] Matt Corgan commented on HBASE-6162: Jesse - thanks. I understand now and agree

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-15 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13295812#comment-13295812 ] Matt Corgan commented on HBASE-6162: No worries stack - i just wasn't sure of the best

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-14 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13294848#comment-13294848 ] Ted Yu commented on HBASE-6162: --- Patch looks good. {code} +++

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-14 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13294871#comment-13294871 ] Hadoop QA commented on HBASE-6162: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-14 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13294918#comment-13294918 ] Hadoop QA commented on HBASE-6162: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-14 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13295308#comment-13295308 ] Hudson commented on HBASE-6162: --- Integrated in HBase-TRUNK #3025 (See

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-14 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13295379#comment-13295379 ] Hudson commented on HBASE-6162: --- Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #54 (See

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-13 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13294820#comment-13294820 ] Ted Yu commented on HBASE-6162: --- From

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-07 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290821#comment-13290821 ] stack commented on HBASE-6162: -- Yeah, lets undo these dependencies. TestConst we agreed

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-07 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13291053#comment-13291053 ] stack commented on HBASE-6162: -- @Matt Makes sense. I'd have seen that if I looked closer.

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290290#comment-13290290 ] Matt Corgan commented on HBASE-6162: Stack - based on your recent maven wrangling,

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290305#comment-13290305 ] Zhihong Ted Yu commented on HBASE-6162: --- Hadoop QA didn't run any test. See

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290525#comment-13290525 ] Zhihong Ted Yu commented on HBASE-6162: --- The patch produces a lot of compilation

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290531#comment-13290531 ] stack commented on HBASE-6162: -- Ok. Lets leave comparators as they are for now. Can break

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290556#comment-13290556 ] stack commented on HBASE-6162: -- When hbase-server compiles its tests, it does not include

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread Jesse Yates (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290682#comment-13290682 ] Jesse Yates commented on HBASE-6162: @stack - yup, that's what I was going on about in

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-06 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13290744#comment-13290744 ] Matt Corgan commented on HBASE-6162: I think we are finally starting to grasp the

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-05 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13289515#comment-13289515 ] stack commented on HBASE-6162: -- Who is 'clarence'? Seems whack that hbase-commons needs to

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-05 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13289526#comment-13289526 ] Matt Corgan commented on HBASE-6162: haha - he is the late Clarence Clemons from the

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-05 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13289530#comment-13289530 ] stack commented on HBASE-6162: -- bq. Do you know if that is a fixable problem? It adds some

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-05 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13289716#comment-13289716 ] Lars Hofhansl commented on HBASE-6162: -- Can we remove all the comparators from

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-05 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13289729#comment-13289729 ] stack commented on HBASE-6162: -- That would make some sense Lars. Move

[jira] [Commented] (HBASE-6162) Move KeyValue to hbase-common module

2012-06-05 Thread Matt Corgan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13289808#comment-13289808 ] Matt Corgan commented on HBASE-6162: I started going down that path, but i believe it