Hi Okay se we do not see this in IDEA - its after all a smarter editor ;)
On Wed, Mar 15, 2017 at 10:26 AM, Zoran Regvart <zo...@regvart.com> wrote: > Hi, > for instance in BeanLookupUsingJndiRegistryIssueTest.java:34[1], the > warrning I get in Eclipse is: > > "Resource leak: 'camel' is never closed" > > this is because the resource leak detection code in IDE is looking for > close(), but we have stop() > > zoran > > [1] > https://github.com/apache/camel/blob/master/camel-core/src/test/java/org/apache/camel/component/bean/BeanLookupUsingJndiRegistryIssueTest.java#L34 > > On Wed, Mar 15, 2017 at 10:01 AM, Claus Ibsen <claus.ib...@gmail.com> wrote: >> Hi >> >> Is there an example you can point we can open in our IDEs to see ? >> >> >> On Wed, Mar 15, 2017 at 9:49 AM, Zoran Regvart <zo...@regvart.com> wrote: >>> Hi, >>> just giving the discussion on the PR#1537[1], CAMEL-11011[2] more >>> visibility here if anyone would like to comment. >>> The downside I found was that the IDE issues a warning about >>> 'Potential resource leak'. I don't know if this would be reason enough >>> to revert this? >>> >>> zoran >>> >>> [1] https://github.com/apache/camel/pull/1537 >>> [2] https://issues.apache.org/jira/browse/CAMEL-11011 >>> -- >>> Zoran Regvart >> >> >> >> -- >> Claus Ibsen >> ----------------- >> http://davsclaus.com @davsclaus >> Camel in Action 2: https://www.manning.com/ibsen2 > > > > -- > Zoran Regvart -- Claus Ibsen ----------------- http://davsclaus.com @davsclaus Camel in Action 2: https://www.manning.com/ibsen2
