[accumulo] branch master updated: Add javadoc explaining parameter behavior. Closes #1619 (#1626)

[mmiller]

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 <mmil...@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