ctubbsii commented on a change in pull request #607: ACCUMULO-4813 Add ability
to provide a load plan for bulk import
URL: https://github.com/apache/accumulo/pull/607#discussion_r216857135
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
##########
@@ -630,28 +631,41 @@ void load()
/**
* @since 2.0.0
*/
- interface ImportExecutorOptions extends ImportSourceOptions {
+ interface ImportDestinationOptions extends ImportFinalOptions {
+
+ /**
+ * Load files in the directory to the row ranges specified in the plan.
The plan should contain
+ * at least one entry for every file in the directory.
+ */
+ ImportFinalOptions usingPlan(LoadPlan service);
Review comment:
I'm not sure I understand the "ImportFinalOptions" naming. What does "Final"
mean in this context?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
