<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
On 06/02/2010 06:58 AM, James Bowes wrote:
<blockquote cite="mid:20100602105816.GA26940@tiny" type="cite">
  <pre wrap="">On Tue, Jun 01, 2010 at 04:55:11PM -0400, Justin Harris wrote:
&lt;snip&gt; 
  </pre>
  <blockquote type="cite">
    <blockquote type="cite">
      <pre wrap="">Since we are using Guice, it does make sense that we would use it to
create a singleton of the Config, but how does that affect the custom
modules we load in the Config to configure Guice? Is this a chicken
and the egg problem?
      </pre>
    </blockquote>
    <pre wrap="">
Quite possibly, maybe we are overloading the Config class too much?  One possible
solution is to split the config into:

1) Config to get Guice up and running
2) Config component that can be injected into other objects
    </pre>
  </blockquote>
  <pre wrap="">
I'd prefer to just instantiate the config twice rather than complicate
things to work around something that only happens once.

Or, we go whole hog and make classes for each configurable part of
candlepin.

  </pre>
  <blockquote type="cite">
    <blockquote type="cite">
      <pre wrap="">
      </pre>
      <blockquote type="cite">
        <pre wrap="">What do you guys think?  Am I overreacting?  :)
        </pre>
      </blockquote>
      <pre wrap="">
See above, and static's aren't always bad.
      </pre>
    </blockquote>
  </blockquote>
  <pre wrap="">
new Config() sticks out with all the other guiceyness, so I'm all for
changing that. I think static makes sense for a config map, however, as
in the non-testing code, you wouldn't want to be altering any of those
values.
  </pre>
</blockquote>
<br>
&nbsp; Typically, in a unix Daemon, kill -1 means reload your config.&nbsp; So,
you would want to reload at that point, possibly by just creating a new
instance.<br>
<blockquote cite="mid:20100602105816.GA26940@tiny" type="cite">
  <pre wrap="">
-James
  </pre>
  <pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
candlepin mailing list
<a class="moz-txt-link-abbreviated" href="mailto:candlepin@lists.fedorahosted.org">candlepin@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/mailman/listinfo/candlepin">https://fedorahosted.org/mailman/listinfo/candlepin</a>
  </pre>
</blockquote>
<br>
</body>
</html>