Opened 13 years ago

Closed 13 years ago

#1866 closed task (fixed)

Refine FDO join shortcut implementation

Reported by: jng Owned by:
Priority: high Milestone: 2.4
Component: Server Version:
Severity: critical Keywords:
Cc: External ID:

Description

It seems I have been barking up the wrong tree somewhat. For some reasons, I was led to believe the new FDO join APIs were somehow baked into the FdoIExtendedSelect command, thus the wrong assumption that FDO join support was limited to the SupportsJoins() capability and support for the FdoIExtendedSelect command

This is false, the required join bits (SetAlias(), GetJoinCriteria()) actually exist in the basic FdoISelect command, so we only need to check the SupportsJoins() capability before apply the additional shortcut checks.

As a result, the ExtendedSelectCommand that's been implemented is totally not required and can be removed as part of this cleanup.

These changes will make the FDO join shortcut also work with the SQL Server provider.

Change History (1)

comment:1 by jng, 13 years ago

Resolution: fixed
Status: newclosed

Fixed with r6251

Note: See TracTickets for help on using tickets.