Would this be from the Thrift calls only?  (i.e. not the
ZookeeperClusterStatus object?)  The console uses the
ZookeeperClusterStatus object to get online/offline shards and controllers
from ZK.

Chris


On Wed, Jul 16, 2014 at 12:27 PM, Tim Williams <[email protected]> wrote:

> One challenge in getting Blur to operate off of random ports is with
> the shard servers.  Currently as a shard server comes up it registers
> itself in the onlineshardserver path as an ephemeral node and again as
> a persistent node on the registered-nodes path.
>
> The problem is with that persistent node - because its name is
> port-based, as shard servers get restarted, we'd see the registered
> nodes list grow artificially.  It turns out that internally the only
> [meaningful] place this is used is in the TopCommand.
>
> Rather than craft some clever reliable naming alternative, I'd like to
> propose just getting rid of the method all together?  Given we're more
> capable of dynamic growth now, its seems pointless anyways?  We could
> change the TopCommand to simply display info about online servers.
> I'm not sure what to do about actually ripping it out - as I don't see
> a great way to deprecate it other than simply return onlineShardsList.
>
> Thoughts?
>
> --tim
>

Reply via email to