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.
Fixed with r6251