<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi all,<br>
    <br>
    I'd like to raise several concerns with the CDI-26 resolution that I
    either forgot to raise before or were lost in the process.<br>
    <br>
    1) Bean discovery in SE<br>
    <br>
    I was under the impression that the task to define bean discovery
    for SE was postponed post EDR1 yet the PR for CDI-26 that has been
    merged defines bean discovery in SE explicitly
<a class="moz-txt-link-freetext" href="https://github.com/johnament/cdi/commit/a112489f248ab9074da4d0a81a28abc67f8cdbe5#diff-ffe540480772deae967ea309fa5f3976R44">https://github.com/johnament/cdi/commit/a112489f248ab9074da4d0a81a28abc67f8cdbe5#diff-ffe540480772deae967ea309fa5f3976R44</a><br>
    <br>
    I am concerned about the way it is defined currently as it requires
    that the CDI implementation eagerly loads/scans each and every class
    found on the classpath during initialization. Due to performance
    implications of this I am convinced that this is not the desired
    behavior. It may be useful to support this for some use-cases with
    e.g. a special container mode but I doubt this should be the default
    behavior for CDI in SE. Let's not forget to fix this.<br>
    <br>
    2)
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    CDIProvider.isInitialized()<br>
    <br>
    First of all, good job on removing the constraints, preventing
    multiple parallel container support, from the API. One missing piece
    seems to be CDIProvider.isInitialized(). The JavaDoc says: "
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    Determines whether or not this CDIProvider has been initialized or
    not"<br>
    <br>
    My understanding is that it is supposed to indicate whether a CDI
    object is in initialized state yet or whether it has been shut down.
    If my understanding is correct then this method should probably be
    moved to the CDI class instead. Due to the possible 1-to-n mapping
    between CDI and CDIProvider it's not correct to have this method on
    CDIProvider<br>
    <br>
    Jozef<br>
  </body>
</html>