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) {