You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/05/20 14:33:54 UTC

[GitHub] [accumulo] keith-turner edited a comment on pull request #1614: Create max tablets property in new bulk import

keith-turner edited a comment on pull request #1614:
URL: https://github.com/apache/accumulo/pull/1614#issuecomment-631512130


   I like the suggestion for a table prop, that aligns more closely with the user experience rather than how its implemented.
   
   The check on the client side is nice for fail fast, but we may want a check on the server side to ensure correctness.  However the server side check is really hard to test, if there is a client side check.
   
   Are there any plans to make the old bulk import code respect this property?


----------------------------------------------------------------
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:
users@infra.apache.org