Stefan Manegold wrote:
> On Sat, Dec 13, 2008 at 03:50:22PM +0000, Martin Kersten wrote:
>> Update of /cvsroot/monetdb/clients/src/mapiclient
>> In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv11819/mapiclient
>>
>> Modified Files:
>> MapiClient.mx
>> Log Message:
>> Start removing the default choice. It should not be an option
>> any longer.
>> Preferrably, we should remove the -g altogether.
>>
>>
>> U MapiClient.mx
>> Index: MapiClient.mx
>> ===================================================================
>> RCS file: /cvsroot/monetdb/clients/src/mapiclient/MapiClient.mx,v
>> retrieving revision 1.130
>> retrieving revision 1.131
>> diff -u -d -r1.130 -r1.131
>> --- MapiClient.mx 5 Dec 2008 14:57:32 -0000 1.130
>> +++ MapiClient.mx 13 Dec 2008 15:50:20 -0000 1.131
>> @@ -1287,7 +1287,6 @@
>> stream_printf(toConsole, "\\a - disable auto commit\n");
>> }
>> if (mode == SQL || mode == XQUERY) {
>> - stream_printf(toConsole, "\\G - switch to algebra frontend\n");
>> stream_printf(toConsole, "\\g - switch to old frontend\n");
>
> hm, this means that you tell users only how to switch to the old frontend,
> but once they have done so, they (now) won't get told (anymore) how to
> switch back, again ...
Yes, I would even go as far as completely dropping/hiding this option
for SQL.
The sole reason not to do it with this check-in was that Mtest and friends
may need a change too.
The owners can propagate that, otherwise I will continue....checking in.
> ... maybe not that handy --- and for XQuery, the "old" backend is both
> still
> available and still required for some functionality ...
The real question is if new users need to be aware of this 'backward'
switch/functionality. Existing users may be supported for a limited time
to avoid failing scripts.
Reduction of the options is preferred, no knobs!
> Hence, IMHO
> (1) "-G" / "-g" and "\G" / "\g" make only sense as pairs, i.e., to switch
> back and forth, i.e., removing one does not make (much) sense; and
> (2) XQuery still requires both for the time being.
>
> Thus, if we don't need either for SQL any more, we should remove the
> documentation (and functionality) of both switches only for SQL, but
> not for
> XQuery --- and if we do so, we should do this everywhere, i.e., not only
> with mclient, but also with Mserver/mserver5, MonetDB.conf/monetdb5.conf,
> Mtest.py and the wbe site --- well, lacking branches, the latter can
> only be
> done once we have a new feature release in about 3 month from now ...
>
> Stefan
>
>
>> }
>> if (mode == XQUERY) {
>> @@ -1709,7 +1708,6 @@
>> fprintf(stderr, " -r nr | --rows=nr for pagination\n");
>> fprintf(stderr, " -w nr | --width=nr for pagination\n");
>> fprintf(stderr, " -D | --dump create an SQL dump\n");
>> - fprintf(stderr, " -G | --algebra use algebra frontend\n");
>> fprintf(stderr, " -g | --no-algebra use old frontend\n");
>>
>>
>> fprintf(stderr, "\nXQuery specific options\n");
>>
>>
>> ------------------------------------------------------------------------------
>> SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas,
>> Nevada.
>> The future of the web can't happen without you. Join us at MIX09 to help
>> pave the way to the Next Web now. Learn more and register at
>> http://ad.doubleclick.net/clk;208669438;135v03038;i?http://2009.visitmix.com/
>> _______________________________________________
>> Monetdb-checkins mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
>>
>>
>
>------------------------------------------------------------------------------ SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas, Nevada. The future of the web can't happen without you. Join us at MIX09 to help pave the way to the Next Web now. Learn more and register at http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/ _______________________________________________ Monetdb-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/monetdb-developers
