Issue Details (XML | Word | Printable)

Key: DERBY-2223
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Dyre Tjeldvoll
Reporter: Dyre Tjeldvoll
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Derby

Let BasePage.fetchFieldFromSlot use the special single-col FetchDescriptor ctor

Created: 09/Jan/07 10:13 AM   Updated: 11/Jan/07 09:36 AM
Return to search
Component/s: Miscellaneous
Affects Version/s: 10.2.2.0
Fix Version/s: 10.3.1.4

Time Tracking:
Not Specified

File Attachments:
  Size
File Licensed for inclusion in ASF works derby-2223.v1.diff 2007-01-09 10:16 AM Dyre Tjeldvoll 0.8 kB
File Licensed for inclusion in ASF works derby-2223.v1.stat 2007-01-09 10:16 AM Dyre Tjeldvoll 0.2 kB

Resolution Date: 11/Jan/07 08:54 AM


 Description  « Hide
BasePage.fetchFieldFromSlot needs to construct a FetchDescriptor with single column. To do this it creates a FormatableBitSet that identifies the correct column and then passes this bitset to the FetchDescriptor constructor. But FetchDescriptor has a special constructor for the single column case, that doesn't need a bitset. Using this constructor instead makes the code simpler, and presumably more efficient as well.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Dyre Tjeldvoll added a comment - 09/Jan/07 10:17 AM
Attached a patch for this. Junit tests and derbyall passes.

Kristian Waagan added a comment - 09/Jan/07 10:42 AM
Patch looks sound.
I'm starting a test run and will commit in a day or two if the test runs without errors.

Kristian Waagan added a comment - 11/Jan/07 08:54 AM
No errors seen in test run.
Patch 'derby-2223.v1.diff ' committed with revision 495159.