accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubbsii <>
Subject [GitHub] accumulo pull request: ACCUMULO-3602 BatchScanner optimization for...
Date Wed, 08 Apr 2015 22:23:27 GMT
Github user ctubbsii commented on the pull request:
    > Gentle reminder that you're looking a gift horse in the mouth with this.
    Perhaps, but isn't that the purpose of a code review? To my knowledge, nobody has expressed
any ill-will or animosity. I, for one, am appreciative that somebody has taken the time to
investigate this. Comprehensive feedback and discussion, and alternative proposals, should
not be construed as judgmental or lacking gratitude, but simply as attempts to flesh out the
eventual solution to be merged.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message