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 2021/07/07 17:27:09 UTC

[GitHub] [accumulo] milleruntime commented on pull request #2191: Refactor LogSorter to use config

milleruntime commented on pull request #2191:
URL: https://github.com/apache/accumulo/pull/2191#issuecomment-875790269


   > The change looks fine. My only concern about this change is that it doesn't fully fix the issue. As far as I can tell, the configuration that is being passed in is always the system config, and never the per-table configuration, in spite of per-table properties being read from it later on.
   
   Correct. It is only an improvement of what is there. This doesn't fix https://github.com/apache/accumulo/issues/2187


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

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org