<div dir="ltr">Ooooooh, reverse that again, it could be cyclic, even though that would make so sense, the system does not enforce it.  Definitely need the accumulator to avoid an infinite loop.  Set two users to report to each other and see if it explodes.  ;)</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 30, 2013 at 10:14 AM, Adam Heinz <span dir="ltr"><<a href="mailto:amh@metricwise.net" target="_blank">amh@metricwise.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="im">On Tue, Apr 30, 2013 at 10:12 AM, Adam Heinz <span dir="ltr"><<a href="mailto:amh@metricwise.net" target="_blank">amh@metricwise.net</a>></span> wrote:<br></div><div class="gmail_extra">
<div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">You should add some sort of accumulator to getReportingUsers so that you don't recur for userids you have already checked.</div>

</blockquote><div><br></div></div><div>Scratch this comment, we're walking a directed acyclic graph; redundancy is impossible.</div></div></div></div>
</blockquote></div><br></div>