accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mmil...@apache.org
Subject [accumulo] branch master updated: Add javadoc explaining parameter behavior. Closes #1619 (#1626)
Date Tue, 16 Jun 2020 15:04:17 GMT
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
     new 8fce661  Add javadoc explaining parameter behavior. Closes #1619 (#1626)
8fce661 is described below

commit 8fce661b72a1ac05705500df243d29a0b0cda379
Author: Mike Miller <mmiller@apache.org>
AuthorDate: Tue Jun 16 11:03:13 2020 -0400

    Add javadoc explaining parameter behavior. Closes #1619 (#1626)
---
 core/src/main/java/org/apache/accumulo/core/data/LoadPlan.java | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/org/apache/accumulo/core/data/LoadPlan.java b/core/src/main/java/org/apache/accumulo/core/data/LoadPlan.java
index 8b8c7a9..81514c0 100644
--- a/core/src/main/java/org/apache/accumulo/core/data/LoadPlan.java
+++ b/core/src/main/java/org/apache/accumulo/core/data/LoadPlan.java
@@ -160,7 +160,8 @@ public class LoadPlan {
    */
   public interface Builder {
     /**
-     * Specify the row range where a file should be loaded.
+     * Specify the row range where a file should be loaded. Note that whether the startRow
parameter
+     * is inclusive or exclusive is determined by the {@link RangeType} parameter.
      *
      * @param fileName
      *          this should not be a path. Only a file name because loads are expected to
happen
@@ -169,7 +170,8 @@ public class LoadPlan {
     Builder loadFileTo(String fileName, RangeType rangeType, Text startRow, Text endRow);
 
     /**
-     * Specify the row range where a file should be loaded.
+     * Specify the row range where a file should be loaded. Note that whether the startRow
parameter
+     * is inclusive or exclusive is determined by the {@link RangeType} parameter.
      *
      * @param fileName
      *          this should not be a path. Only a file name because loads are expected to
happen
@@ -178,7 +180,8 @@ public class LoadPlan {
     Builder loadFileTo(String fileName, RangeType rangeType, byte[] startRow, byte[] endRow);
 
     /**
-     * Specify the row range where a file should be loaded.
+     * Specify the row range where a file should be loaded. Note that whether the startRow
parameter
+     * is inclusive or exclusive is determined by the {@link RangeType} parameter.
      *
      * @param fileName
      *          this should not be a path. Only a file name because loads are expected to
happen


Mime
View raw message