+ ConfigBeanProxy proxy = (ConfigBeanProxy)(event.getSource());
+ Config config;
+ if(proxy instanceof MonitoringService) {
+ config = (Config)proxy.getParent();
+ } else {
+ config = (Config)proxy.getParent().getParent();
+ }
+
+ String name = config.getName();
+ // We think that DAS's config is always server-config.
+ if(name.equalsIgnoreCase("server-config")) {
Is is true that the DAS's config is required to be named "server-config"?
Does other code depend on that?
Are config names case-independent? I wouldn't think so.
Why is the MonitoringService special? Shouldn't it just go up the parent
chain until it gets to something that's "instanceof Config"?
lvsongping wrote on 09/18/13 00:39:
> Hi, Byron:
>
> Cc: dev team:
>
>
>
> I have fixed the issue about the GLASFISH-20696which have been reviewed by my
> team and also ran the QL tests and all of the tests are passed after I have
> applied the revision of GLASFISH-20696.
>
>
> Please review my changes and give feedback. If I don't hear from you in 3weeks,
> by 10/7, I will go ahead and commit the changes.
>
>
>
> Thanks a lot!
>
>
>
> Regards
>
> Jeremy Lv
>
> --------------------------------------------------
>
> Lv Songping
>
> Software Division II
>
> Development Department I
>
> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST)
>
> ADDR.: No.6 Wenzhu Road, Software Avenue,
>
> Nanjing, 210012, China
>
> TEL : +86+25-86630566-8307
>
> COINS: 7998-8307
>
> FAX : +86+25-83317685
>
> MAIL : lvsongping_at_cn.fujitsu.com <mailto:lvsongping_at_cn.fujitsu.com>
>
>
>