dev@glassfish.java.net

Re: (QL failure) Issue 6001 has been re-assigned to SQE-Test

From: Jan Luehe <Jan.Luehe_at_Sun.COM>
Date: Tue, 09 Sep 2008 18:42:34 -0700

Kedar,

On 09/ 9/08 04:57 PM, Kedar Mhaswade wrote:
>
>
> Jan,
>
> A quick check on http-service reveals that there are no attributes on
> it. So, till Jerome/I get time to investigate this, can you just ignore
> the CHANGE event on element http-service to resolve this immediate
> problem?

Yes, the attached diffs make QL pass for me consistently.

I'd like for Amy to have the last word on this, though, to see if any of
our unit
tests might fail with this temporary workaround in place.

Thanks,

Jan



Index: web/web-glue/src/main/java/com/sun/enterprise/web/reconfig/HttpServiceConfigListener.java
===================================================================
--- web/web-glue/src/main/java/com/sun/enterprise/web/reconfig/HttpServiceConfigListener.java (revision 22504)
+++ web/web-glue/src/main/java/com/sun/enterprise/web/reconfig/HttpServiceConfigListener.java (working copy)
@@ -165,6 +165,9 @@
                         container.configureHttpProtocol(httpService);
                     } else if (t instanceof HttpFileCache) {
                         container.configureFileCache(httpService);
+ } else if (t instanceof HttpService) {
+ // Workaround for Issue 6001
+ return null;
                     }
                     container.updateHttpService(httpService);
                 } catch (LifecycleException le) {