Re: [Monetdb-developers] [Monetdb-checkins] clients/src/mapiclient MapiClient.mx, , 1.130, 1.131
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 ... ... maybe not that handy --- and for XQuery, the "old" backend is both still available and still required for some functionality ... 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;13503038;i?http://2009.visitmix.com/ _______________________________________________ Monetdb-checkins mailing list Monetdb-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | 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 |
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 Monetdb-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
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? 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. 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 Monetdb-checkins@lists.sourceforge.net 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 Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | 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 |
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 Monetdb-checkins@lists.sourceforge.net 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 Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | 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: Niels.Nes@cwi.nl
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 Monetdb-checkins@lists.sourceforge.net 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 Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | 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: Niels.Nes@cwi.nl
------------------------------------------------------------------------------ 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 Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | 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 |
participants (3)
-
Martin Kersten -
Niels Nes -
Stefan Manegold