Chris, Thanks for the explanation. I think this one may be more that I can handle right now so I am going to annotate on the ticket that I am going to stop work on it and I will find something else.
Don ________________________________ From: Christopher <[email protected]> Sent: Wednesday, May 15, 2019 1:12 PM To: accumulo-dev Subject: Re: question about issue 1123 [External Email] ---------------------------------------------------------------------- Removing the observers requires rewriting the entire mechanism for handling configuration changes that observers are currently handling. It's not a simple replacement of the existing methods to an equivalent set of methods. It would have to be rewritten to leverage ZooCache's ability to track changes with a counter, and to reload when that counter is different. On Tue, May 14, 2019 at 9:54 PM d resnik <[email protected]> wrote: > > Devs, > I am trying to work on this ticket, but I am getting stuck. > Remove observers from configuration · Issue #1123 · apache/accumulo > > > | > | > | > | | | > > | > > | > | > | | > Remove observers from configuration · Issue #1123 · apache/accumulo > > https://urldefense.proofpoint.com/v2/url?u=https-3A__issues.apache.org_jira_browse_ACCUMULO-2D4822&d=DwIFaQ&c=Ff2GokX4A_6S-cy_JRNEiQ&r=7sXp0We1OQQKRSGyjz35VSGU6op68dwzfY3c89DwDgBombhjVkfUnmcvB18wjBra&m=aCI6CI4b4W3gOW1tWR1ZgaPhh49dWAUxQgcmN2KMR88&s=nPs4bgN3cwSK581yy9r_dUgztkOXMhrKup7JQeQASIQ&e= > | > > | > > | > > > Specifically, what should I do in NamespaceConfWatcher in the case on line 94 > > case None: > > switch (event.getState()) { > > case Expired: > > ServerConfigurationFactory.expireAllTableObservers(); > break; > Which ends up trying to call expireAllObservers() on all the > TableConfigurations. This method is no longer available because > TableConfiguration now extends AccumuloConfigruation in my branch. > What is the equivalent to expireAllTableObservers() and/or > expireAllObservers() if I remove all the Observer refs? Or does this case > even need to be there without the Observer classes. > Any ideas would be appreciated. > Thanks, > Don ________________________________ The preceding message (including attachments) is covered by the Electronic Communication Privacy Act, 18 U.S.C. sections 2510-2512, is intended only for the person or entity to which it is addressed, and may contain information that is confidential, protected by attorney-client or other privilege, or otherwise protected from disclosure by law. If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited. Please reply to the sender that you have received the message in error and destroy the original message and all copies.
