This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a commit to branch branch-2.5
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.5 by this push:
     new 9b33ab6feb5 HBASE-28651 Reformat the javadoc for CellChunkMap (#5977)
9b33ab6feb5 is described below

commit 9b33ab6feb5bc5bfb7d56371105a1e681061e04e
Author: Duo Zhang <zhang...@apache.org>
AuthorDate: Tue Jun 11 22:03:35 2024 +0800

    HBASE-28651 Reformat the javadoc for CellChunkMap (#5977)
    
    Signed-off-by: Yi Mei <me...@apache.org>
    (cherry picked from commit 580361820f6d678ef8044dc2b1a8b00b1b30c5ee)
---
 .../hadoop/hbase/regionserver/CellChunkMap.java    | 32 ++++++++++++++--------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CellChunkMap.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CellChunkMap.java
index 8a5f28c2870..e4bfcf05ab2 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CellChunkMap.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CellChunkMap.java
@@ -27,18 +27,28 @@ import org.apache.yetus.audience.InterfaceAudience;
 
 /**
  * CellChunkMap is an array of serialized representations of Cell (pointing to 
Chunks with full Cell
- * data) and can be allocated both off-heap and on-heap. CellChunkMap is a 
byte array (chunk)
- * holding all that is needed to access a Cell, which is actually saved on 
another deeper chunk. Per
- * Cell we have a reference to this deeper byte array B (chunk ID, integer), 
offset in bytes in B
- * (integer), length in bytes in B (integer) and seqID of the cell (long). In 
order to save
- * reference to byte array we use the Chunk's ID given by ChunkCreator. The 
CellChunkMap memory
- * layout on chunk A relevant to a deeper byte array B, holding the actual 
cell data: < header >
- * <--------------- first Cell -----------------> <-- second Cell ...
+ * data) and can be allocated both off-heap and on-heap.
+ * <p>
+ * CellChunkMap is a byte array (chunk) holding all that is needed to access a 
Cell, which is
+ * actually saved on another deeper chunk. Per Cell we have a reference to 
this deeper byte array B
+ * (chunk ID, integer), offset in bytes in B (integer), length in bytes in B 
(integer) and seqID of
+ * the cell (long). In order to save reference to byte array we use the 
Chunk's ID given by
+ * ChunkCreator.
+ * <p>
+ * The CellChunkMap memory layout on chunk A relevant to a deeper byte array 
B, holding the actual
+ * cell data:
+ *
+ * <pre>
+ *
+ * < header > <---------------     first Cell     -----------------> <-- 
second Cell ...
  * 
---------------------------------------------------------------------------------------
 ...
- * integer | integer | integer | integer | long | 4 bytes | 4 bytes | 4 bytes 
| 4 bytes | 8 bytes |
- * ChunkID | chunkID of | offset in B | length of | sequence | ... of this | 
chunk B with | where
- * Cell's | Cell's | ID of | chunk A | Cell data | data starts | data in B | 
the Cell |
+ *  integer  | integer      | integer      | integer     | long     |
+ *  4 bytes  | 4 bytes      | 4 bytes      | 4 bytes     | 8 bytes  |
+ *  ChunkID  | chunkID of   | offset in B  | length of   | sequence |          
...
+ *  of this  | chunk B with | where Cell's | Cell's      | ID of    |
+ *  chunk A  | Cell data    | data starts  | data in B   | the Cell |
  * 
---------------------------------------------------------------------------------------
 ...
+ * </pre>
  */
 @InterfaceAudience.Private
 public class CellChunkMap extends CellFlatMap {
@@ -71,7 +81,7 @@ public class CellChunkMap extends CellFlatMap {
     }
   }
 
-  /*
+  /**
    * To be used by base (CellFlatMap) class only to create a sub-CellFlatMap 
Should be used only to
    * create only CellChunkMap from CellChunkMap
    */

Reply via email to