Done --- hopefully completely and consistently in code, documentation (usgae
measages) and stable test outputs.

Stefan

On Sun, Dec 14, 2008 at 05:36:17PM +0100, Niels Nes wrote:
> On Sun, Dec 14, 2008 at 05:32:13PM +0100, Stefan Manegold wrote:
> > Niels,
> > 
> > can you please confirm that any functionality to explicitly switch between
> > the "old" (direct-binary) and "new" (Algebra) version of SQL not required
> > any more?
> > On the server side, "--set sql_debug=16384" used to enable the Algebra
> > version before it has become default last week.
> > Am I right that (a) "--set sql_debug=16384" does not (have to) do this job
> > any more and (b) there is no new/respective functionality to switch back to
> > the old version?
> algebra is default and doesn't require sql_debug=16384 anymore.
> The old default is removed from the sources!
> 
> > 
> > In that case, we should indeed consistently and completely remove all
> > back-end switching functionality *for SQL* (only) from the code (server,
> > client, testing) as weel as from the respective documentation, website and
> > usage messages.
>  From the server this is done. For -g/G should be dependend on -lxquery
> then I guess.
> 
> Niels
> > 
> > Stefan
> > 
> > On Sat, Dec 13, 2008 at 07:11:01PM +0100, Martin Kersten wrote:
> > > 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
> > > 
> > > 
> > 
> > -- 
> > | Dr. Stefan Manegold | mailto:[email protected] |
> > | CWI,  P.O.Box 94079 | http://www.cwi.nl/~manegold/  |
> > | 1090 GB Amsterdam   | Tel.: +31 (20) 592-4212       |
> > | The Netherlands     | Fax : +31 (20) 592-4312       |
> 
> -- 
> 
> Niels Nes, Centre for Mathematics and Computer Science (CWI)
> Kruislaan 413, 1098 SJ Amsterdam, The Netherlands
> room C0.02,  phone ++31 20 592-4098, fax ++31 20 592-4312
> url: http://www.cwi.nl/~niels   e-mail: [email protected]
> 
> ------------------------------------------------------------------------------
> 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
> 
> 
> 
> 

-- 
| Dr. Stefan Manegold | mailto:[email protected] |
| CWI,  P.O.Box 94079 | http://www.cwi.nl/~manegold/  |
| 1090 GB Amsterdam   | Tel.: +31 (20) 592-4212       |
| The Netherlands     | Fax : +31 (20) 592-4312       |

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

Reply via email to