|
A B made changes - 18/Nov/07 01:38 AM
A B made changes - 18/Nov/07 01:39 AM
A B made changes - 06/Dec/07 08:00 PM
A B made changes - 06/Dec/07 08:01 PM
A B made changes - 06/Dec/07 08:02 PM
A B, your patch looks good from what I can tell.
I've started suites.All but don't expect any problems from this patch. My suites.All did show some errors, but I haven't had the time to investigate wether it's related to your patch or not as of now.
I get the same errors both with and without your patch A B, so it most likely not related. Since your tests passed without issues, I'm +1 for commit.
Thank you for the review, Thomas. I committed the patch to trunk with svn # 603659:
http://svn.apache.org/viewvc?rev=603659&view=rev
A B made changes - 12/Dec/07 04:18 PM
A B made changes - 12/Dec/07 04:19 PM
There was a glitch in the first patch: when recovering the cost for the sort avoidance plan, we need to look at the sort avoidance costs (not the default "best access" costs). Attaching d3214_followup_1.patch, which I committed with svn # 604513:
http://svn.apache.org/viewvc?rev=604513&view=rev
A B made changes - 15/Dec/07 08:52 PM
I just noticed that this patch was not ported to 10.3, even though the issue that exposes it (
derbyall and suites.All ran cleanly with this patch using 10.3 jars and ibm142. So I committed the changes to 10.3 with svn # 615076:
URL: http://svn.apache.org/viewvc?rev=615076&view=rev Adding 10.3 to the Fix In list. If no problems arise in 10.3 tinderbox over the next couple of days, I plan to close this issue.
A B made changes - 24/Jan/08 11:20 PM
A B made changes - 31/Jan/08 04:40 PM
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
DERBY-3023.