Problem with ResourceChangeScanner with UrlResources in 5.5 - rules-dev - Jboss List Archives

2025

January

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

Problem with ResourceChangeScanner with UrlResources in 5.5

Dependencies upgraded: cxf,...

Drools 6.0 - Slides from the...

Demian Calcaprina

Sunday, 25 August 2013 Sun, 25 Aug '13

6:25 p.m.

Hey Guys.Yesterday I found an issue in drools 5.5. Basically, the ResourceChangeScanner is removing my URL resource from the KnowledgeAgent. I created a ticket with the description, the cause and a pull request. https://issues.jboss.org/browse/DROOLS-240 It seems to be quite critical, so you may want to merge to 5.6 branch. Thanks! Demian

Attachments:

attachment.html (text/html — 604 bytes)

Reply

Show replies by date

Mark Proctor

Monday, 9 September Mon, 9 Sep

6:39 a.m.

We don't normally merge fixes, without unit tests to catch regressions - can you add a test? Mark On 25 Aug 2013, at 17:25, Demian Calcaprina <calcacuervo(a)gmail.com> wrote:

Hey Guys.Yesterday I found an issue in drools 5.5. Basically, the ResourceChangeScanner is removing my URL resource from the KnowledgeAgent. I created a ticket with the description, the cause and a pull request. https://issues.jboss.org/browse/DROOLS-240 It seems to be quite critical, so you may want to merge to 5.6 branch. Thanks! Demian _______________________________________________ rules-dev mailing list rules-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-dev

Reply

Demian Calcaprina

2:40 p.m.

Hey Mark, of course. This is not easy to create a unit test for it I think, as it need some guvnor (or some url with auth). Are there some tests which are more like integration tests where I can add it? In the ticket I added a main class where I could reproduce it directly using the same java.net.* classes. https://issues.jboss.org/secure/attachment/12364295/UrlTest.java Thanks, Demian On Mon, Sep 9, 2013 at 1:39 AM, Mark Proctor <mproctor(a)codehaus.org> wrote:

We don't normally merge fixes, without unit tests to catch regressions - can you add a test? Mark On 25 Aug 2013, at 17:25, Demian Calcaprina <calcacuervo(a)gmail.com> wrote: Hey Guys.Yesterday I found an issue in drools 5.5. Basically, the ResourceChangeScanner is removing my URL resource from the KnowledgeAgent. I created a ticket with the description, the cause and a pull request. https://issues.jboss.org/browse/DROOLS-240 It seems to be quite critical, so you may want to merge to 5.6 branch. Thanks! Demian _______________________________________________ rules-dev mailing list rules-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-dev _______________________________________________ rules-dev mailing list rules-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/rules-dev

Reply

4136

days inactive

4151

days old

rules-dev@lists.jboss.org

Manage subscription

2 comments

2 participants

tags (0)

participants (2)

Demian Calcaprina
Mark Proctor