Hi Dave, Everything seems alright.
Thank you for applying the patch. Now I don't have to patch Jena in my projects manually. Bye, Tilo On Wed, 20 Mar 2013 08:52:41 +0000 Dave Reynolds <[email protected]> wrote: > Hi Tilo, > > The patch looks good to me. As you say there is a space cost but the > trade-off seems reasonable. > > Andy has created issue JENA-412 to track this request. I've checked > and applied the patch. If you give it 24 hours for the snapshot > builds to propagate then you can check whether 2.10.1-SNAPSHOT > correctly includes your patch. If so let us know so we can close the > issue. > > Many thanks for your contribution. > > Dave > > [1] https://issues.apache.org/jira/browse/JENA-412 > > On 17/03/13 07:24, Tilo Fischer wrote: > > I would like to like to submit a patch which speeds up forward rule > > execution in the RETE engine. > > It uses only slightly more memory, but reduces the time needed for > > my dataset of about 100.000 triples from beyond measuring to about 5 > > minutes. > > It passes all tests so far. Am I missing something? > > > > Tilo Fischer > > >
