accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Vines" <>
Subject Re: Review Request 14995: ACCUMULO-1679 - Purges TFile by rolling the two static calls into BCFile
Date Mon, 28 Oct 2013 22:09:27 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 28, 2013, 10:09 p.m.)

Review request for accumulo.


Updated with patch which removed Chunk as well

Bugs: ACCUMULO-1679

Repository: accumulo


Dirt simple fix which moves the 2 static variables and methods into BCFile. I'm pretty sure
we can't go about renaming those constants, but I could be mistaken. General trivialness of
this fix is why I'm review boarding it - it seems too easy.

Diffs (updated)

  core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/ 2d9f6af 
  core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/ a075d87 
  core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/ e89bb40 
  core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/ f2cb326 
  core/src/main/java/org/apache/accumulo/core/file/rfile/bcfile/ d5b0a1b 




John Vines

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message