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

alexey 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 7d1f3fb  [C++ client] doxygenized C++ client API in 
columnar_scan_batch.h
7d1f3fb is described below

commit 7d1f3fbb40984b6917e6e76f252bf6a0924fd70d
Author: hahao <[email protected]>
AuthorDate: Thu May 14 15:28:21 2020 -0700

    [C++ client] doxygenized C++ client API in columnar_scan_batch.h
    
    This patch documents the missing parameters for APIs in 
columnar_scan_batch.h
    to satisfy the warnings from doxygen.
    
    Change-Id: Ifb8b927bf55ff181b9923411c35729eed8bafa08
    Reviewed-on: http://gerrit.cloudera.org:8080/15918
    Reviewed-by: Grant Henke <[email protected]>
    Tested-by: Kudu Jenkins
    Reviewed-by: Alexey Serbin <[email protected]>
---
 src/kudu/client/columnar_scan_batch.h | 41 ++++++++++++++++++++++++-----------
 1 file changed, 28 insertions(+), 13 deletions(-)

diff --git a/src/kudu/client/columnar_scan_batch.h 
b/src/kudu/client/columnar_scan_batch.h
index 924c162..953ce8b 100644
--- a/src/kudu/client/columnar_scan_batch.h
+++ b/src/kudu/client/columnar_scan_batch.h
@@ -58,35 +58,50 @@ class KUDU_EXPORT KuduColumnarScanBatch {
 
   /// Get the raw columnar data corresponding to the primitive-typed column 
with index 'idx'.
   ///
-  /// The data is in little-endian packed array format. No alignment or 
padding is guaranteed.
-  /// Space is reserved for all cells regardless of whether they might be null.
-  /// The data stored in a null cell may or may not be zeroed.
-  ///
   /// For variable-length (e.g. STRING, BINARY, VARCHAR) columns, use
   /// GetVariableLengthColumn instead.
   ///
   /// @note The Slice returned is only valid for the lifetime of the 
KuduColumnarScanBatch.
+  ///
+  /// @param [in] idx
+  ///   The column index.
+  /// @param [out] data
+  ///   The data is in little-endian packed array format. No alignment or 
padding is guaranteed.
+  ///   Space is reserved for all cells regardless of whether they might be 
null.
+  ///   The data stored in a null cell may or may not be zeroed.
+  /// @return Operation result status.
   Status GetFixedLengthColumn(int idx, Slice* data) const;
 
   /// Return the variable-length data for the variable-length-typed column 
with index 'idx'.
   ///
-  /// If NumRows() is 0, the 'offsets' array will have length 0. Otherwise, 
this array
-  /// will contain NumRows() + 1 entries, each indicating an offset within the
-  /// variable-length data array returned in 'data'. For each cell with index 
'n',
-  /// offsets[n] indicates the starting offset of that cell, and offsets[n+1] 
indicates
-  /// the ending offset of that cell.
+  /// @param [in] idx
+  ///   The column index.
+  /// @param [out] offsets
+  ///   If NumRows() is 0, the 'offsets' array will have length 0. Otherwise, 
this array
+  ///   will contain NumRows() + 1 entries, each indicating an offset within 
the
+  ///   variable-length data array returned in 'data'. For each cell with 
index 'n',
+  ///   offsets[n] indicates the starting offset of that cell, and 
offsets[n+1] indicates
+  ///   the ending offset of that cell.
+  /// @param [out] data
+  ///   The variable-length data.
+  /// @return Operation result status.
   Status GetVariableLengthColumn(int idx, Slice* offsets, Slice* data) const;
 
   /// Get a bitmap corresponding to the non-null status of the cells in the 
given column.
   ///
-  /// A set bit indicates a non-null cell.
-  /// If the number of rows is not a multiple of 8, the state of the trailing 
bits in the
-  /// bitmap is undefined.
-  ///
   /// It is an error to call this function on a column which is not marked as 
nullable
   /// in the schema.
   ///
   /// @note The Slice returned is only valid for the lifetime of the 
KuduColumnarScanBatch.
+  ///
+  /// @param [in] idx
+  ///   The column index.
+  /// @param [out] data
+  ///   The bitmap corresponding to the non-null status of the cells in the 
given column.
+  ///   A set bit indicates a non-null cell.
+  ///   If the number of rows is not a multiple of 8, the state of the 
trailing bits in the
+  ///   bitmap is undefined.
+  /// @return Operation result status.
   Status GetNonNullBitmapForColumn(int idx, Slice* data) const;
 
  private:

Reply via email to