> Thanks for the patch. I've applied it with modifications to my local
> copy. 

Out of curiosity - what kind of modifications? I ask because if there is 
anything I can do to make the patches more acceptable, I will see to it.

> I didn't commit it yet because I just realized the remote_side 
> argument is stepping on SQLalchemy's toes. I'm not sure I was aware of
> it when I implemented that argument, because, luckily, it has not much
> sense in a ManyToMany relationship. Nevertheless I think it'd be a
> good idea to change its name. What do you (and others if they read
> this) think of: "local_colnames" and "remote_colnames"?

Sounds better for me than *_side, as it better captures the actual 
meaning. "side" can be a lot...

Diez

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"SQLElixir" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/sqlelixir?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to