[fdo-internals] RFC 50 updated (II)

[Haris Kurtagic]

sorry, I somehow managed to send email too early while typing body of it

to continue on point 4)

In one of samples we have in previous conversation we had self-join with
 parent.id = child.id where parent and child are aliases. I think that
both "parent" and "child"  has to be defined on level of join criteria
as aliases of "parcel" class.
Could be even more important now when we can have filter for
join-criteria with sub-select , which again can have join criteria in
it.

Haris

On Fri, Jun 25, 2010 at 12:36 PM, Haris Kurtagic <haris at sl-king.com> wrote:
> Hi Romy,
>
> First of all, thanks for effort in discussing and changing this RFC.
>
> I have few suggestions/questions:
>
> 1) To change FdoSubSelectExpression from FdoExpression to FdoValueExpression
>    It would allow  use of FdoSubSelectExpression in inserts and
> updates as well.
>
>
> 2) FdoJoinCriteriaCollection has functions
>   /// Returns FdoJoinCriteria having the name or NULL.
>          FDO_API FdoJoinCriteria* FindItem(FdoString* name) and
> FDO_API FdoJoinCriteria* GeiItem(FdoString* name)
>
>    Parameter name is to search over join criteria alias or class name or both ?
>    Change name and description of function to be clear what it is ?
>
> 3) What is supposed to be FdoJoinType_None ? implicit join (where) ?
>
> 4) I think I can see problems in join filter implementations.
>    I think it is important that when join criteria is used and class
> aliases are used in filter that those aliases are clearly defined in
> join criteria.
>   Otherwise I believe we will run into difficult and non-consistent
> implementations across providers.
>
>
>   In one of samples we have in previous conversation we had self-join with
>   parent.id = child.id where parent and child are aliases
>
>
> Haris
>
> On Thu, Jun 24, 2010 at 9:29 PM, Romica Dascalescu
> <Romica.Dascalescu at autodesk.com> wrote:
>> Hi,
>>
>> RFC 50 (http://trac.osgeo.org/fdo/wiki/FDORfc50) has been updated to include all comments received and is now ready for review.
>> Added a new capability: SupportsSubSelects() in order to allow applications to detect if a connection supports sub-selects in filters.
>>
>> Please review and provide additional feedback.
>>
>> Thanks,
>> Romy.
>>
>> _______________________________________________
>> fdo-internals mailing list
>> fdo-internals at lists.osgeo.org
>> http://lists.osgeo.org/mailman/listinfo/fdo-internals
>>
>