[
https://issues.apache.org/jira/browse/HBASE-2270?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12838588#action_12838588
]
Jean-Daniel Cryans commented on HBASE-2270:
-------------------------------------------
bq. A funny aside, the 'continue' lines essentially act like goto statements
Yeah that's ugly... although in the WildcardColumnTracker case I'm not sure how
to get rid of them. I'm trying not to change the code too much and it seems
that not continuing would break the behavior. Suggestions welcomed!
> Improve how we handle recursive calls in ExplicitColumnTracker and
> WildcardColumnTracker
> ----------------------------------------------------------------------------------------
>
> Key: HBASE-2270
> URL: https://issues.apache.org/jira/browse/HBASE-2270
> Project: Hadoop HBase
> Issue Type: Improvement
> Reporter: Jean-Daniel Cryans
> Assignee: Jean-Daniel Cryans
> Priority: Minor
> Fix For: 0.20.4, 0.21.0
>
> Attachments: HBASE-2270.patch
>
>
> Ryan was saying in HBASE-2259:
> {quote}
> the variable 'recursive' should really be named something more accurate. the
> continue at the end of the loop should get optimized out, but it looks like
> bad form. also commented out code should not be in the patch, just remove it.
> since this call is also tail recursive, this should be solvable with a
> while(true) and return statements only, no boolean necessary. That would be a
> more straightforward port, so lets do that instead.
> {quote}
> Let's fix ExplicitColumnTracker and WildcardColumnTracker at the same time.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
