accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] keith-turner commented on issue #509: ACCUMULO-3510 Updated scheduler to use comparators for sessions
Date Tue, 05 Jun 2018 18:22:13 GMT
keith-turner commented on issue #509: ACCUMULO-3510 Updated scheduler to use comparators for
sessions
URL: https://github.com/apache/accumulo/pull/509#issuecomment-394811050
 
 
   @bbux-atg  thanks for the contribution.  I have one reservation about this change and that
is exposing internal types like Session to user pluggable code.  Developers currently change
internal types as needed and my not be aware that they are breaking plugins.  I think it would
be best to introduce a new type that the pluggable code compares.  If you would like I can
make a pass at introducing this type, just let me know.

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


With regards,
Apache Git Services

Mime
View raw message