You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ad...@apache.org on 2019/10/09 18:33:13 UTC

[kudu] 01/02: [common] Fix comment of RowProjector

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

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

commit 2618ef370d2848fc949bc6f8da01e74ae871f26e
Author: lingbin <li...@gmail.com>
AuthorDate: Wed Oct 9 18:43:59 2019 +0800

    [common] Fix comment of RowProjector
    
    Change-Id: Ie41f41b9bf0b7b2b2eb80d1b4f548c83a421bee5
    Reviewed-on: http://gerrit.cloudera.org:8080/14401
    Tested-by: Kudu Jenkins
    Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
 src/kudu/common/row.h | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/kudu/common/row.h b/src/kudu/common/row.h
index 45090fc..846ba02 100644
--- a/src/kudu/common/row.h
+++ b/src/kudu/common/row.h
@@ -133,8 +133,8 @@ inline Status CopyRow(const RowType1 &src_row, RowType2 *dst_row, ArenaType *dst
 //    In this case the default value of the projection column will be used.
 //
 // Example:
-//  RowProjector projector.
-//  projector.Init(base_schema, projection);
+//  RowProjector projector(&base_schema, &projection);
+//  projector.Init();
 //  projector.ProjectRow(row_a, &row_b, &row_b_arena);
 class RowProjector {
  public:
@@ -299,7 +299,7 @@ class DeltaProjector {
   }
 
  private:
-  friend class ::kudu::Schema;
+  friend class Schema;
 
   Status ProjectBaseColumn(size_t proj_col_idx, size_t base_col_idx) {
     base_cols_mapping_[proj_col_idx] = base_col_idx;
@@ -379,7 +379,6 @@ inline Status RelocateIndirectDataToArena(RowType *row, ArenaType *dst_arena) {
   return Status::OK();
 }
 
-
 class ContiguousRowHelper {
  public:
   static size_t null_bitmap_size(const Schema& schema) {