Re: [Monetdb-developers] [Monetdb-pf-checkins] pathfinder/tests/StandOff StandOff.py, , 1.10, 1.11
Wouter,
On Tue, Dec 23, 2008 at 10:40:57AM +0000, Wouter Alink wrote:
Update of /cvsroot/monetdb/pathfinder/tests/StandOff In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv29912
Modified Files: StandOff.py Log Message:
- this commit switches StandOff testing towards Algebra-backend
was/is it you intension to force all StandOff testing though the Algebra back-end (added "-A" option for pf; see below), i.e., ignoring/overruling the compile time default / choice (whch is indeed the Algebra back-end) as well as the choice on the Mtest.py command line?
If so, why? Does MPS not longer support StandOff (or v.v.)?
If not, you should remove the "-A" swicth for pf, again.
- observation: order of attributes seems to have changed in some tests, the testoutput has been changed accordingly
Serialization in MonetDB/XQuery has not feature (yet?) to enforce a particular attribute order. The order of attributes is only determined by the very implementation and (physical) order of input data, and can hence change. If it happens to differe between the MPS & ALG back-end for (some) StandOff tests, we could consider approving back-end specific (ALG or MPS) stable output for these tests. (See `Mtest.py --help` and/or http://monetdb.cwi.nl/Development/TestWeb/Mtest/ for details and/or feel free to ask for advice/help.)
Stefan
- the "video/artists" test still fails, due to incorrect order of the elements in the result. this test needs more investigation
U StandOff.py Index: StandOff.py =================================================================== RCS file: /cvsroot/monetdb/pathfinder/tests/StandOff/StandOff.py,v retrieving revision 1.10 retrieving revision 1.11 diff -u -d -r1.10 -r1.11 --- StandOff.py 15 Feb 2008 18:34:11 -0000 1.10 +++ StandOff.py 23 Dec 2008 10:40:55 -0000 1.11 @@ -27,7 +27,7 @@ TSTSRCDIR = os.environ['TSTSRCDIR'] PF = os.environ['PF']
-CALL = '%s -b "%s.xq" | %s --set standoff=enabled --dbname=%s "--dbinit=module(pathfinder);"' % (PF,os.path.join(TSTSRCDIR,TST),MSERVER,TSTDB) +CALL = '%s -bA "%s.xq" | %s --set standoff=enabled --dbname=%s "--dbinit=module(pathfinder);"' % (PF,os.path.join(TSTSRCDIR,TST),MSERVER,TSTDB)
import sys, time Mlog = "\n%s %s\n\n" % (time.strftime('# %H:%M:%S >',time.localtime(time.time())), CALL)
Monetdb-pf-checkins mailing list Monetdb-pf-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-pf-checkins
was/is it you intension to force all StandOff testing though the Algebra back-end (added "-A" option for pf; see below), i.e., ignoring/overruling the compile time default / choice (whch is indeed the Algebra back-end) as well as the choice on the Mtest.py command line?
If so, why? Does MPS not longer support StandOff (or v.v.)?
If not, you should remove the "-A" swicth for pf, again.
it was my intention to switch to the algebra backend, but from the testweb I probably incorrectly assumed that the milprint-summer version was still the default for pf. (i thought i saw only the artists query failing when i looked at the testweb this morning) I will remove the -A, again. The StandOff aces should still work with MPS.
- observation: order of attributes seems to have changed in some tests, the testoutput has been changed accordingly
Serialization in MonetDB/XQuery has not feature (yet?) to enforce a particular attribute order. The order of attributes is only determined by the very implementation and (physical) order of input data, and can hence change. If it happens to differe between the MPS & ALG back-end for (some) StandOff tests, we could consider approving back-end specific (ALG or MPS) stable output for these tests. (See `Mtest.py --help` and/or http://monetdb.cwi.nl/Development/TestWeb/Mtest/ for details and/or feel free to ask for advice/help.)
I assumed (again probably incorrectly) that the milprint-summer version is deprecated. I'll create separate test-results.
Thanks for observing my errors. Wouter
Ah...
I figured out why now... I hadn't seen your recent changes to main.c.
Thanks, Wouter
2008/12/23 Wouter Alink wouter.alink@gmail.com:
was/is it you intension to force all StandOff testing though the Algebra back-end (added "-A" option for pf; see below), i.e., ignoring/overruling the compile time default / choice (whch is indeed the Algebra back-end) as well as the choice on the Mtest.py command line?
If so, why? Does MPS not longer support StandOff (or v.v.)?
If not, you should remove the "-A" swicth for pf, again.
it was my intention to switch to the algebra backend, but from the testweb I probably incorrectly assumed that the milprint-summer version was still the default for pf. (i thought i saw only the artists query failing when i looked at the testweb this morning) I will remove the -A, again. The StandOff aces should still work with MPS.
- observation: order of attributes seems to have changed in some tests, the testoutput has been changed accordingly
Serialization in MonetDB/XQuery has not feature (yet?) to enforce a particular attribute order. The order of attributes is only determined by the very implementation and (physical) order of input data, and can hence change. If it happens to differe between the MPS & ALG back-end for (some) StandOff tests, we could consider approving back-end specific (ALG or MPS) stable output for these tests. (See `Mtest.py --help` and/or http://monetdb.cwi.nl/Development/TestWeb/Mtest/ for details and/or feel free to ask for advice/help.)
I assumed (again probably incorrectly) that the milprint-summer version is deprecated. I'll create separate test-results.
Thanks for observing my errors. Wouter
On Tue, Dec 23, 2008 at 12:10:27PM +0100, Stefan Manegold wrote:
Wouter,
On Tue, Dec 23, 2008 at 10:40:57AM +0000, Wouter Alink wrote:
Update of /cvsroot/monetdb/pathfinder/tests/StandOff In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv29912
Modified Files: StandOff.py Log Message:
- this commit switches StandOff testing towards Algebra-backend
was/is it you intension to force all StandOff testing though the Algebra back-end (added "-A" option for pf; see below), i.e., ignoring/overruling the compile time default / choice (whch is indeed the Algebra back-end) as well as the choice on the Mtest.py command line?
If so, why? Does MPS not longer support StandOff (or v.v.)?
If not, you should remove the "-A" swicth for pf, again.
- observation: order of attributes seems to have changed in some tests, the testoutput has been changed accordingly
Serialization in MonetDB/XQuery has not feature (yet?) to enforce a particular attribute order. The order of attributes is only determined by the very implementation and (physical) order of input data, and can hence change. If it happens to differe between the MPS & ALG back-end for (some) StandOff tests, we could consider approving back-end specific (ALG or MPS) stable output for these tests. (See `Mtest.py --help` and/or http://monetdb.cwi.nl/Development/TestWeb/Mtest/ for details and/or feel free to ask for advice/help.)
"Xmas present":
- made StandOff tests (again) use the default back-end, respectively the one chosen when calling Mtest.py (via -G/--algebra or -g/--no-algebra)
- reverted default stable output to represent attribute order as produced with MPS back-end - added ALG specific stable output with ALG produced attribute order (Mtest.py does not (yet??) offer to have MPS-specific stable output)
Merry Chirstmas! ;-)
Stefan
Stefan
- the "video/artists" test still fails, due to incorrect order of the elements in the result. this test needs more investigation
U StandOff.py Index: StandOff.py =================================================================== RCS file: /cvsroot/monetdb/pathfinder/tests/StandOff/StandOff.py,v retrieving revision 1.10 retrieving revision 1.11 diff -u -d -r1.10 -r1.11 --- StandOff.py 15 Feb 2008 18:34:11 -0000 1.10 +++ StandOff.py 23 Dec 2008 10:40:55 -0000 1.11 @@ -27,7 +27,7 @@ TSTSRCDIR = os.environ['TSTSRCDIR'] PF = os.environ['PF']
-CALL = '%s -b "%s.xq" | %s --set standoff=enabled --dbname=%s "--dbinit=module(pathfinder);"' % (PF,os.path.join(TSTSRCDIR,TST),MSERVER,TSTDB) +CALL = '%s -bA "%s.xq" | %s --set standoff=enabled --dbname=%s "--dbinit=module(pathfinder);"' % (PF,os.path.join(TSTSRCDIR,TST),MSERVER,TSTDB)
import sys, time Mlog = "\n%s %s\n\n" % (time.strftime('# %H:%M:%S >',time.localtime(time.time())), CALL)
Monetdb-pf-checkins mailing list Monetdb-pf-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-pf-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 |
Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers
Ho ho ho ho merry x mas:-)
2008/12/24, Stefan Manegold Stefan.Manegold@cwi.nl:
On Tue, Dec 23, 2008 at 12:10:27PM +0100, Stefan Manegold wrote:
Wouter,
On Tue, Dec 23, 2008 at 10:40:57AM +0000, Wouter Alink wrote:
Update of /cvsroot/monetdb/pathfinder/tests/StandOff In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv29912
Modified Files: StandOff.py Log Message:
- this commit switches StandOff testing towards Algebra-backend
was/is it you intension to force all StandOff testing though the Algebra back-end (added "-A" option for pf; see below), i.e., ignoring/overruling the compile time default / choice (whch is indeed the Algebra back-end) as well as the choice on the Mtest.py command line?
If so, why? Does MPS not longer support StandOff (or v.v.)?
If not, you should remove the "-A" swicth for pf, again.
- observation: order of attributes seems to have changed in some tests, the testoutput has been changed accordingly
Serialization in MonetDB/XQuery has not feature (yet?) to enforce a particular attribute order. The order of attributes is only determined by the very implementation and (physical) order of input data, and can hence change. If it happens to differe between the MPS & ALG back-end for (some) StandOff tests, we could consider approving back-end specific (ALG or MPS) stable output for these tests. (See `Mtest.py --help` and/or http://monetdb.cwi.nl/Development/TestWeb/Mtest/ for details and/or feel free to ask for advice/help.)
"Xmas present":
made StandOff tests (again) use the default back-end, respectively the one chosen when calling Mtest.py (via -G/--algebra or -g/--no-algebra)
reverted default stable output to represent attribute order as produced with MPS back-end
added ALG specific stable output with ALG produced attribute order (Mtest.py does not (yet??) offer to have MPS-specific stable output)
Merry Chirstmas! ;-)
Stefan
Stefan
- the "video/artists" test still fails, due to incorrect order of the elements in the result. this test needs more investigation
U StandOff.py Index: StandOff.py =================================================================== RCS file: /cvsroot/monetdb/pathfinder/tests/StandOff/StandOff.py,v retrieving revision 1.10 retrieving revision 1.11 diff -u -d -r1.10 -r1.11 --- StandOff.py 15 Feb 2008 18:34:11 -0000 1.10 +++ StandOff.py 23 Dec 2008 10:40:55 -0000 1.11 @@ -27,7 +27,7 @@ TSTSRCDIR = os.environ['TSTSRCDIR'] PF = os.environ['PF']
-CALL = '%s -b "%s.xq" | %s --set standoff=enabled --dbname=%s "--dbinit=module(pathfinder);"' % (PF,os.path.join(TSTSRCDIR,TST),MSERVER,TSTDB) +CALL = '%s -bA "%s.xq" | %s --set standoff=enabled --dbname=%s "--dbinit=module(pathfinder);"' % (PF,os.path.join(TSTSRCDIR,TST),MSERVER,TSTDB)
import sys, time Mlog = "\n%s %s\n\n" % (time.strftime('# %H:%M:%S
',time.localtime(time.time())), CALL)
Monetdb-pf-checkins mailing list Monetdb-pf-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-pf-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 |
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 |
Monetdb-developers mailing list Monetdb-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-developers
participants (3)
-
Arjen P. de Vries -
Stefan Manegold -
Wouter Alink