[GitHub] [accumulo] ctubbsii edited a comment on issue #1333: LRUCache is Evicting Too Many CachedBlocks

[GitBox <gi...@apache.org>]

ctubbsii edited a comment on issue #1333: LRUCache is Evicting Too Many CachedBlocks
URL: https://github.com/apache/accumulo/pull/1333#issuecomment-550145368
 
 
   @belugabehr This issue has crept a bit from its starting point. I'd like to refocus with a question: Do you have a concrete end goal in mind here, and is that goal important to you?
   
   I ask if it's important to you because my impression is that you're trying to get more involved, but that these specific code changes have stalled (on the side of the reviewers) due to being high-risk and low-reward (because the code is complicated and the goal has gotten fuzzy). I understand that stall can make you feel discouraged, which we definitely don't want to do.
   
   So, if it is important to you, perhaps there's some way you can define your overall goal more concretely so that others can understand and so that you can make progress incrementally towards it. But, if this area of the code is not that important to you, then it might be a good idea to abandon the work because the risk-reward tradeoff isn't worth it, and for us to help encourage you contributing elsewhere. I'm okay with either direction, but currently, I worry this issue isn't headed in a direction that makes the best use of your time.

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


With regards,
Apache Git Services