Johannes Janssen wrote:

> I propose a patch for resolve(), which makes to major changes:
>
> User defined resolve function

I like this particular aspect of the patch, in theory. It allows for the 
user to define the entity resolution process to their heart's content.

> Resolving the module path

I don't particularly like this idea. I think that the current approach 
is simpler and more easy to understand. If we accept the patch for the 
user-defined resolve function, you could (in theory) implement this 
behavior in your own project if you wanted to.

> For backwards compatibility you might give users the
> possibility to switch back manually to old behavior and give them a
> deprecation warning (for a few versions if there are to be more;)).

This is another reason I dislike the second part of your patch.

Gaetan, what are your thoughts? Anyone else have any opinions on the 
subject?

-- 
Jonathan LaCour
http://cleverdevil.org

--~--~---------~--~----~------------~-------~--~----~
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