[
https://issues.apache.org/jira/browse/PHOENIX-1151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14089860#comment-14089860
]
James Taylor commented on PHOENIX-1151:
---------------------------------------
I guess I misunderstood, as I thought the code wasn't necessary anymore. It'd
be good to get rid of that call (since it's the only one and sounds like it's
not the "right way of doing things" anymore), and replace it with the "right
way to do it".
Do we need to wrap every RegionScanner in the delegate like the current code is
doing (even if the coprocessor isn't the one that handles the scan coming
through)? Or can we exit early and not do that bit if isRegionObserverFor(scan)
returns false? Or do we no longer need to wrap it?
Maybe you can just send me a pull request, [~jesse_yates]?
> Remove unnecessary tracing code
> -------------------------------
>
> Key: PHOENIX-1151
> URL: https://issues.apache.org/jira/browse/PHOENIX-1151
> Project: Phoenix
> Issue Type: Bug
> Reporter: James Taylor
> Attachments: PHOENIX-1151.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.2#6252)
