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
-- | 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 |
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
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
-- | 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 |
Ho ho ho ho merry x mas:-)
2008/12/24, Stefan Manegold
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
-- Verzonden vanaf mijn mobiele apparaat ==================================================================== CWI, room C1.16 Centre for Mathematics and Computer Science Science Park 123 Email: Arjen.de.Vries@cwi.nl 1098 XG Amsterdam tel: +31-(0)20-5924306 The Netherlands fax: +31-(0)20-5924312 ===================== http://www.cwi.nl/~arjen/ ====================
participants (3)
-
Arjen P. de Vries -
Stefan Manegold -
Wouter Alink