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

zhangyifan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 78ee89e  Fix some typo
78ee89e is described below

commit 78ee89ef6218038d05e004f341092650741f8132
Author: xlwh <[email protected]>
AuthorDate: Mon Feb 21 21:05:41 2022 +0800

    Fix some typo
    
    Change-Id: I8bd4a3f2840b22e1421f7cc7a6031419d298ab3b
    Reviewed-on: http://gerrit.cloudera.org:8080/18259
    Tested-by: Kudu Jenkins
    Reviewed-by: Yifan Zhang <[email protected]>
---
 src/kudu/cfile/binary_dict_block.h   | 2 +-
 src/kudu/cfile/binary_plain_block.h  | 4 ++--
 src/kudu/cfile/binary_prefix_block.h | 6 +++---
 src/kudu/cfile/index_block.h         | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/kudu/cfile/binary_dict_block.h 
b/src/kudu/cfile/binary_dict_block.h
index 60f4643..fa07fc5 100644
--- a/src/kudu/cfile/binary_dict_block.h
+++ b/src/kudu/cfile/binary_dict_block.h
@@ -176,7 +176,7 @@ class BinaryDictBlockDecoder final : public BlockDecoder {
   Slice data_;
   bool parsed_;
 
-  // Dictionary block decoder
+  // Dictionary block decoder.
   BinaryPlainBlockDecoder* dict_decoder_;
 
   std::unique_ptr<BlockDecoder> data_decoder_;
diff --git a/src/kudu/cfile/binary_plain_block.h 
b/src/kudu/cfile/binary_plain_block.h
index edd4784..25cfc84 100644
--- a/src/kudu/cfile/binary_plain_block.h
+++ b/src/kudu/cfile/binary_plain_block.h
@@ -25,7 +25,7 @@
 // Strings:
 //   raw strings that were written
 // Offsets:  [pointed to by offsets_pos]
-//   gvint-encoded offsets pointing to the beginning of each string
+//   gvint-encoded offsets pointing to the beginning of each string.
 #ifndef KUDU_CFILE_BINARY_PLAIN_BLOCK_H
 #define KUDU_CFILE_BINARY_PLAIN_BLOCK_H
 
@@ -93,7 +93,7 @@ class BinaryPlainBlockBuilder final : public BlockBuilder {
   size_t end_of_data_offset_;
   size_t size_estimate_;
 
-  // Offsets of each entry, relative to the start of the block
+  // Offsets of each entry, relative to the start of the block.
   std::vector<uint32_t> offsets_;
 
   bool finished_;
diff --git a/src/kudu/cfile/binary_prefix_block.h 
b/src/kudu/cfile/binary_prefix_block.h
index fa5e180..ca6e1ad 100644
--- a/src/kudu/cfile/binary_prefix_block.h
+++ b/src/kudu/cfile/binary_prefix_block.h
@@ -44,7 +44,7 @@ namespace cfile {
 struct WriterOptions;
 
 // Encoding for data blocks of binary data that have common prefixes.
-// This encodes in a manner similar to LevelDB (prefix coding)
+// This encodes in a manner similar to LevelDB (prefix coding).
 class BinaryPrefixBlockBuilder final : public BlockBuilder {
  public:
   explicit BinaryPrefixBlockBuilder(const WriterOptions *options);
@@ -72,7 +72,7 @@ class BinaryPrefixBlockBuilder final : public BlockBuilder {
   faststring buffer_;
   faststring last_val_;
 
-  // Restart points, offsets relative to start of block
+  // Restart points, offsets relative to start of block.
   std::vector<uint32_t> restarts_;
 
   int val_count_;
@@ -114,7 +114,7 @@ class BinaryPrefixBlockDecoder final : public BlockDecoder {
   }
 
   // Minimum length of a header.
-  // Currently one group of varints for an empty block, so minimum is 5 bytes
+  // Currently one group of varints for an empty block, so minimum is 5 bytes.
   static const size_t kMinHeaderSize = 5;
 
  private:
diff --git a/src/kudu/cfile/index_block.h b/src/kudu/cfile/index_block.h
index cb51bcd..045c010 100644
--- a/src/kudu/cfile/index_block.h
+++ b/src/kudu/cfile/index_block.h
@@ -105,7 +105,7 @@ class IndexBlockReader {
   // a new block.
   //
   // Note: this does not copy the data, so the slice must
-  // remain valid for the lifetime of the reader (or until the next Parse call)
+  // remain valid for the lifetime of the reader (or until the next Parse 
call).
   Status Parse(const Slice &data);
 
   size_t Count() const;

Reply via email to