[jira] [Commented] (HBASE-4476) Compactions must fail if column tracker gets columns out of order

2011-09-30 Thread Mikhail Bautin (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13118429#comment-13118429
 ] 

Mikhail Bautin commented on HBASE-4476:
---

This patch passes all unit tests.

 Compactions must fail if column tracker gets columns out of order
 -

 Key: HBASE-4476
 URL: https://issues.apache.org/jira/browse/HBASE-4476
 Project: HBase
  Issue Type: Bug
Reporter: Mikhail Bautin
Assignee: Mikhail Bautin
 Fix For: 0.92.0, 0.94.0


 We found this in ScanWildcardColumnTracker:
 // new col  oldcol
 // if (cmp  0) {
 // WARNING: This means that very likely an edit for some other family
 // was incorrectly stored into the store for this one. Continue, but
 // complain.
 LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
   into a column actually smaller than the previous column:  +
 This went under the radar in our dark launch cluster when a column family 
 name was misspelled first, but then was renamed by renaming directories in 
 the HBase storage directory tree. We ended up with inconsistent data, but 
 compactions still succeeded most of the time, likely discarding part of input 
 data.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4476) Compactions must fail if column tracker gets columns out of order

2011-09-30 Thread jirapos...@reviews.apache.org (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13118436#comment-13118436
 ] 

jirapos...@reviews.apache.org commented on HBASE-4476:
--


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2050/#review2233
---

Ship it!


- Ted


On 2011-09-29 23:45:46, Mikhail Bautin wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/2050/
bq.  ---
bq.  
bq.  (Updated 2011-09-29 23:45:46)
bq.  
bq.  
bq.  Review request for hbase and Kannan Muthukkaruppan.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  We found this in ScanWildcardColumnTracker:
bq.  
bq.  // new col  oldcol
bq.  // if (cmp  0) {
bq.  // WARNING: This means that very likely an edit for some other family
bq.  // was incorrectly stored into the store for this one. Continue, but
bq.  // complain.
bq.  LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
bq.  into a column actually smaller than the previous column:  +
bq.  
bq.  This went under the radar in our dark launch cluster when a column family 
name was misspelled first, but then was renamed by renaming directories in 
the HBase storage directory tree. We ended up with inconsistent data, but 
compactions still succeeded most of the time, likely discarding part of input 
data.
bq.  
bq.  
bq.  This addresses bug HBASE-4476.
bq.  https://issues.apache.org/jira/browse/HBASE-4476
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ColumnTracker.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanQueryMatcher.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanWildcardColumnTracker.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestExplicitColumnTracker.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestScanWildcardColumnTracker.java
 1176613 
bq.  
bq.  Diff: https://reviews.apache.org/r/2050/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Run unit tests.
bq.  Will also push to a 5-node dev cluster.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Mikhail
bq.  
bq.



 Compactions must fail if column tracker gets columns out of order
 -

 Key: HBASE-4476
 URL: https://issues.apache.org/jira/browse/HBASE-4476
 Project: HBase
  Issue Type: Bug
Reporter: Mikhail Bautin
Assignee: Mikhail Bautin
 Fix For: 0.92.0, 0.94.0


 We found this in ScanWildcardColumnTracker:
 // new col  oldcol
 // if (cmp  0) {
 // WARNING: This means that very likely an edit for some other family
 // was incorrectly stored into the store for this one. Continue, but
 // complain.
 LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
   into a column actually smaller than the previous column:  +
 This went under the radar in our dark launch cluster when a column family 
 name was misspelled first, but then was renamed by renaming directories in 
 the HBase storage directory tree. We ended up with inconsistent data, but 
 compactions still succeeded most of the time, likely discarding part of input 
 data.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4476) Compactions must fail if column tracker gets columns out of order

2011-09-30 Thread jirapos...@reviews.apache.org (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13118456#comment-13118456
 ] 

jirapos...@reviews.apache.org commented on HBASE-4476:
--


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2050/#review2235
---

Ship it!


- Michael


On 2011-09-29 23:45:46, Mikhail Bautin wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/2050/
bq.  ---
bq.  
bq.  (Updated 2011-09-29 23:45:46)
bq.  
bq.  
bq.  Review request for hbase and Kannan Muthukkaruppan.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  We found this in ScanWildcardColumnTracker:
bq.  
bq.  // new col  oldcol
bq.  // if (cmp  0) {
bq.  // WARNING: This means that very likely an edit for some other family
bq.  // was incorrectly stored into the store for this one. Continue, but
bq.  // complain.
bq.  LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
bq.  into a column actually smaller than the previous column:  +
bq.  
bq.  This went under the radar in our dark launch cluster when a column family 
name was misspelled first, but then was renamed by renaming directories in 
the HBase storage directory tree. We ended up with inconsistent data, but 
compactions still succeeded most of the time, likely discarding part of input 
data.
bq.  
bq.  
bq.  This addresses bug HBASE-4476.
bq.  https://issues.apache.org/jira/browse/HBASE-4476
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ColumnTracker.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanQueryMatcher.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanWildcardColumnTracker.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestExplicitColumnTracker.java
 1176613 
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestScanWildcardColumnTracker.java
 1176613 
bq.  
bq.  Diff: https://reviews.apache.org/r/2050/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Run unit tests.
bq.  Will also push to a 5-node dev cluster.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Mikhail
bq.  
bq.



 Compactions must fail if column tracker gets columns out of order
 -

 Key: HBASE-4476
 URL: https://issues.apache.org/jira/browse/HBASE-4476
 Project: HBase
  Issue Type: Bug
Reporter: Mikhail Bautin
Assignee: Mikhail Bautin
 Fix For: 0.92.0, 0.94.0


 We found this in ScanWildcardColumnTracker:
 // new col  oldcol
 // if (cmp  0) {
 // WARNING: This means that very likely an edit for some other family
 // was incorrectly stored into the store for this one. Continue, but
 // complain.
 LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
   into a column actually smaller than the previous column:  +
 This went under the radar in our dark launch cluster when a column family 
 name was misspelled first, but then was renamed by renaming directories in 
 the HBase storage directory tree. We ended up with inconsistent data, but 
 compactions still succeeded most of the time, likely discarding part of input 
 data.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4476) Compactions must fail if column tracker gets columns out of order

2011-09-30 Thread Hudson (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13118588#comment-13118588
 ] 

Hudson commented on HBASE-4476:
---

Integrated in HBase-0.92 #35 (See 
[https://builds.apache.org/job/HBase-0.92/35/])
HBASE-4476 Compactions must fail if column tracker gets columns out of order

stack : 
Files : 
* /hbase/branches/0.92/CHANGES.txt
* 
/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/regionserver/ColumnTracker.java
* 
/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/regionserver/ScanQueryMatcher.java
* 
/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/regionserver/ScanWildcardColumnTracker.java
* 
/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
* 
/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
* 
/hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/regionserver/TestExplicitColumnTracker.java
* 
/hbase/branches/0.92/src/test/java/org/apache/hadoop/hbase/regionserver/TestScanWildcardColumnTracker.java


 Compactions must fail if column tracker gets columns out of order
 -

 Key: HBASE-4476
 URL: https://issues.apache.org/jira/browse/HBASE-4476
 Project: HBase
  Issue Type: Bug
Reporter: Mikhail Bautin
Assignee: Mikhail Bautin
 Fix For: 0.92.0, 0.94.0

 Attachments: HBASE-4476_v2_2011-09-27_17_38_48.patch


 We found this in ScanWildcardColumnTracker:
 // new col  oldcol
 // if (cmp  0) {
 // WARNING: This means that very likely an edit for some other family
 // was incorrectly stored into the store for this one. Continue, but
 // complain.
 LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
   into a column actually smaller than the previous column:  +
 This went under the radar in our dark launch cluster when a column family 
 name was misspelled first, but then was renamed by renaming directories in 
 the HBase storage directory tree. We ended up with inconsistent data, but 
 compactions still succeeded most of the time, likely discarding part of input 
 data.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4476) Compactions must fail if column tracker gets columns out of order

2011-09-30 Thread Hudson (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13118631#comment-13118631
 ] 

Hudson commented on HBASE-4476:
---

Integrated in HBase-TRUNK #2274 (See 
[https://builds.apache.org/job/HBase-TRUNK/2274/])
HBASE-4476 Compactions must fail if column tracker gets columns out of order

stack : 
Files : 
* /hbase/trunk/CHANGES.txt
* 
/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ColumnTracker.java
* 
/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanQueryMatcher.java
* 
/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanWildcardColumnTracker.java
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
* 
/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
* 
/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestExplicitColumnTracker.java
* 
/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestScanWildcardColumnTracker.java


 Compactions must fail if column tracker gets columns out of order
 -

 Key: HBASE-4476
 URL: https://issues.apache.org/jira/browse/HBASE-4476
 Project: HBase
  Issue Type: Bug
Reporter: Mikhail Bautin
Assignee: Mikhail Bautin
 Fix For: 0.92.0, 0.94.0

 Attachments: HBASE-4476_v2_2011-09-27_17_38_48.patch


 We found this in ScanWildcardColumnTracker:
 // new col  oldcol
 // if (cmp  0) {
 // WARNING: This means that very likely an edit for some other family
 // was incorrectly stored into the store for this one. Continue, but
 // complain.
 LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
   into a column actually smaller than the previous column:  +
 This went under the radar in our dark launch cluster when a column family 
 name was misspelled first, but then was renamed by renaming directories in 
 the HBase storage directory tree. We ended up with inconsistent data, but 
 compactions still succeeded most of the time, likely discarding part of input 
 data.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4476) Compactions must fail if column tracker gets columns out of order

2011-09-29 Thread jirapos...@reviews.apache.org (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13117749#comment-13117749
 ] 

jirapos...@reviews.apache.org commented on HBASE-4476:
--


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2050/
---

(Updated 2011-09-29 23:45:46.477105)


Review request for hbase and Kannan Muthukkaruppan.


Changes
---

Linking to HBASE-4476.


Summary
---

We found this in ScanWildcardColumnTracker:

// new col  oldcol
// if (cmp  0) {
// WARNING: This means that very likely an edit for some other family
// was incorrectly stored into the store for this one. Continue, but
// complain.
LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
into a column actually smaller than the previous column:  +

This went under the radar in our dark launch cluster when a column family name 
was misspelled first, but then was renamed by renaming directories in the 
HBase storage directory tree. We ended up with inconsistent data, but 
compactions still succeeded most of the time, likely discarding part of input 
data.


This addresses bug HBASE-4476.
https://issues.apache.org/jira/browse/HBASE-4476


Diffs
-

  
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ColumnTracker.java
 1176613 
  
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanQueryMatcher.java
 1176613 
  
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/ScanWildcardColumnTracker.java
 1176613 
  
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
 1176613 
  
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
 1176613 
  
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestExplicitColumnTracker.java
 1176613 
  
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/TestScanWildcardColumnTracker.java
 1176613 

Diff: https://reviews.apache.org/r/2050/diff


Testing
---

Run unit tests.
Will also push to a 5-node dev cluster.


Thanks,

Mikhail



 Compactions must fail if column tracker gets columns out of order
 -

 Key: HBASE-4476
 URL: https://issues.apache.org/jira/browse/HBASE-4476
 Project: HBase
  Issue Type: Bug
Reporter: Mikhail Bautin
Assignee: Mikhail Bautin
 Fix For: 0.92.0, 0.94.0


 We found this in ScanWildcardColumnTracker:
 // new col  oldcol
 // if (cmp  0) {
 // WARNING: This means that very likely an edit for some other family
 // was incorrectly stored into the store for this one. Continue, but
 // complain.
 LOG.error(ScanWildcardColumnTracker.checkColumn ran  +
   into a column actually smaller than the previous column:  +
 This went under the radar in our dark launch cluster when a column family 
 name was misspelled first, but then was renamed by renaming directories in 
 the HBase storage directory tree. We ended up with inconsistent data, but 
 compactions still succeeded most of the time, likely discarding part of input 
 data.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira