ctubbsii commented on issue #680: URL: https://github.com/apache/accumulo/issues/680#issuecomment-844670745
> In AccumuloClient a method for creating conditional writers that takes no config could be added. We could add an overloaded `createConditionalWriter(String tableName)` that just calls `createConditionalWriter(tableName, new ConditionalWriterConfig())`, but conditional writers seem used infrequently enough that it doesn't really warrant the overload, since this is already pretty easy to do. > Doing this would probably require also adding client properties for the conditional writer. It wouldn't *require* it, but we could make it so `new ConditionalWriterConfig()` gets default values from client properties. How common is it to have an application need to create many different ConditionalWriters with the same ConditionalWriterConfig, though? And, it seems convenient enough to be able to reuse a ConditionalWriterConfig instance and pass it as the second arg in those cases. I'm not sure the advantage to growing client properties over simply reusing the object. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
