Eli Mesika has posted comments on this change. Change subject: engine: Alphanumeric sorting order for db query results (no bz) ......................................................................
Patch Set 1: No score Have talked with Miki and we had concluded to have a meeting on that. BTW If we must have this feature, I still think that we should have a general method of implementing that for all data sources (i.e. Search, query from AD , query from VDSM etc.) If the decision will be to implement only on search, it should be implement like this. for each table T and column C that should support this method: 1) add a column __C with its default as the regexp in this patch 2) add an index on __C for table T 3) change all ORDER BY clause in Search to order by __C optional & recommended: 4) change other SP/Views to use __C in ORDER BY to support also other non-search queries. after that, we still will have 2 open issues ( and probably bugs will opened on that) Non DB queries : a) vdsm queries b) AD/LDAP queries This is the reason why the solution should cover all kinds of queries on the 1st place, if we will implement the search in the DB level as I have described above, we will probably left after a while with at least 3 different mechanisms doing the same logic ..... -- To view, visit http://gerrit.ovirt.org/5079 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ice3ae8b9f897690e777dc53b659359e5cefcd292 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Rami Vaknin <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Rami Vaknin <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
