Re: [Monetdb-developers] [Monetdb-checkins] MonetDB/src/testing Mtest.py.in, , 1.333, 1.334
Stefan Manegold wrote:
Update of /cvsroot/monetdb/MonetDB/src/testing In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv26786/MonetDB/src/testing
Modified Files: Mtest.py.in Log Message:
Added "-a / --algebra" switch to Mtest.py to enable the Algebra version of the SQL compiler (once we have a proper switch for Mserver / mclient, also Pathfinder can use this switch). Well, the option -a does not work while the option --algebra works fine.
the Error: Traceback (most recent call last): File "/ufs/goncalve/scratch/MonetDB/debug/MonetDB/bin/Mtest.py", line 3742, in <module> main(sys.argv) File "/ufs/goncalve/scratch/MonetDB/debug/MonetDB/bin/Mtest.py", line 2912, in main opts, args = monet_options.parse_options(argv[1:], options, Usage, False) File "/ufs/goncalve/scratch/MonetDB/debug/MonetDB/lib64/python2.5/site-packages/MonetDB/monet_options.py", line 211, in parse_options opts, args = getopt.getopt(argv, getopt_short, getopt_long) File "/usr/lib64/python2.5/getopt.py", line 91, in getopt opts, args = do_shorts(opts, args[0][1:], shortopts, args[1:]) File "/usr/lib64/python2.5/getopt.py", line 191, in do_shorts if short_has_arg(opt, shortopts): File "/usr/lib64/python2.5/getopt.py", line 207, in short_has_arg raise GetoptError('option -%s not recognized' % opt, opt) getopt.GetoptError: option -a not recognized Romulo
Via `Mapprove.py -S.Algebra ...`, algebra-specific stable output can be approved.
Also changed the default for template, sql, & geom to use M5 server iso. M4 server.
Index: Mtest.py.in =================================================================== RCS file: /cvsroot/monetdb/MonetDB/src/testing/Mtest.py.in,v retrieving revision 1.333 retrieving revision 1.334 diff -u -d -r1.333 -r1.334 --- Mtest.py.in 11 Jan 2008 10:37:02 -0000 1.333 +++ Mtest.py.in 28 Jan 2008 10:34:14 -0000 1.334 @@ -1211,6 +1211,7 @@ env['TST_OIDS'] = "" env['TST_FIVE'] = "" env['TST_STATIC'] = "" + env['TST_ALGEBRA'] = "" if qOut: tbs = re.compile("^# Compiled for .*/([63][42]bit) with ([63][42])bit OIDs;? (.*) linked", re.MULTILINE) tm = re.compile("^Modules: (.*)$", re.MULTILINE) @@ -1262,6 +1263,9 @@ if par['M5']: env['TST_FIVE'] = "Five" os.environ['TST_FIVE'] = "Five" + if par['ALGEBRA']: + env['TST_ALGEBRA'] = "Algebra" + os.environ['TST_ALGEBRA'] = "Algebra" return rtrn ### GetBitsAndOIDsAndModsAndStatic(env) #
@@ -1375,12 +1379,15 @@ OIDS = env['TST_OIDS'] FIVE = env['TST_FIVE'] STATIC = env['TST_STATIC'] + ALGEBRA = env['TST_ALGEBRA'] if FIVE: FIVE = "(\.Five)?" if STATIC: STATIC = "(\.STATIC)?" + if ALGEBRA: + ALGEBRA = "(\.Algebra)?" dir,file = os.path.split(TST) - outre = re.compile('^'+re.escape(file)+'\.stable\.(?P<tp>out|err)(\.(%s(%s)?|%s(%s)?))?(\.%s)?(\.%s)?%s%s$' % (re.escape(OS), re.escape(RELEASE), re.escape(DIST), re.escape(VERSION), BITS, OIDS, FIVE, STATIC)) + outre = re.compile('^'+re.escape(file)+'\.stable\.(?P<tp>out|err)(\.(%s(%s)?|%s(%s)?))?(\.%s)?(\.%s)?%s%s%s$' % (re.escape(OS), re.escape(RELEASE), re.escape(DIST), re.escape(VERSION), BITS, OIDS, FIVE, STATIC, ALGEBRA)) bestout = besterr = '' for f in os.listdir(dir or os.curdir): res = outre.match(f) @@ -2206,6 +2213,8 @@ Srvr = '%s "--dbinit=%s"' % (Srvr, DBINIT) if SERVER == "SQL": lang="sql" + if par['ALGEBRA']: + Srvr = '%s --set sql_debug=16384' % Srvr if par['M5']: Srvr = '%s --set mal_listing=0 "--dbinit=%s include sql;"' % (Srvr, DBINIT) else: @@ -2534,6 +2543,7 @@ v['OIDS'] = '' v['FIVE'] = '' v['STATIC'] = '' + v['ALGEBRA'] = '' if os.path.isfile(f): r = re.compile('^([A-Z]+) = "(.*)".*$') for l in open(f): @@ -2709,7 +2719,7 @@ }
par = {} -# the default; only used for usage information in case -p/---package has not been given +# the default; only used for usage information in case -p/--package has not been given par['PACKAGE'] = 'monetdb' # try to be "smart" if os.path.exists(os.path.join('CVS', 'Repository')): @@ -2808,6 +2818,8 @@ "use Mserver of MonetDB Version 4 (default with <package> not in {monetdb5, monetdb-datacell})"), ('MonetDB5', '5', 'MonetDB5', None, "use mserver5 of MonetDB Version 5 (default with <package> in {monetdb5, monetdb-datacell})"), + ('algebra', 'a', 'algebra', None, + "use the algebra version of the SQL compiler"), ('recursive', 'r', 'recursive', None, "recurse into subdirectories (implies 'All')"), ('MONETDB_SOURCE', None, 'MONETDB_SOURCE', '<path>', @@ -2872,7 +2884,7 @@ elif THISFILE == 'Mapprove.py': f = os.path.join(GetMonetConfig(par['PACKAGE'],'--prefix'),dftTSTPREF,'.Mapprove.rc') v = ReadMapproveRc(f) - for i in 'BITS', 'OIDS', 'FIVE', 'STATIC': + for i in 'BITS', 'OIDS', 'FIVE', 'STATIC', 'ALGEBRA': if v[i]: v[i] = '[.%s]' % v[i] options = cmd_options + [ @@ -2881,9 +2893,9 @@ "(default: <ext> = 'out' & 'err')"), (None, 'S', 'sys', '<sys>', "approve specific output *.<ext><sys>\n" - "(<sys> = '[.(<OS>[<RELEASE>]|<DIST>[<VERSION>])][.(32|64)bit][.oid(32|64)][.Five][.STATIC]',\n" - "(default: longest match for <sys> = '[.(%s[%s]|%s[%s])]%s%s%s%s')" - % (v['OS'], v['RELEASE'], v['DIST'], v['VERSION'], v['BITS'], v['OIDS'], v['FIVE'], v['STATIC'])), + "(<sys> = '[.(<OS>[<RELEASE>]|<DIST>[<VERSION>])][.(32|64)bit][.oid(32|64)][.Five][.STATIC][.Algebra]',\n" + "(default: longest match for <sys> = '[.(%s[%s]|%s[%s])]%s%s%s%s%s')" + % (v['OS'], v['RELEASE'], v['DIST'], v['VERSION'], v['BITS'], v['OIDS'], v['FIVE'], v['STATIC'], v['ALGEBRA'])), (None, 'f', 'force', None, "force approval of error messages (i.e., lines starting with '!')"), ] @@ -2934,8 +2946,8 @@ if par['M5'] and par['PACKAGE'] == 'monetdb4': ErrXit('"-5/--MonetDB5" conflicts with "--package=monetdb4"!') if not par['M4'] and not par['M5']: - par['M4'] = par['PACKAGE'] not in ['monetdb5', 'monetdb-datacell'] - par['M5'] = par['PACKAGE'] in ['monetdb5', 'monetdb-datacell'] + par['M4'] = par['PACKAGE'] not in ['monetdb5', 'monetdb-datacell', 'monetdb-template', 'monetdb-sql', 'monetdb-geom'] + par['M5'] = par['PACKAGE'] in ['monetdb5', 'monetdb-datacell', 'monetdb-template', 'monetdb-sql', 'monetdb-geom']
config = opts.get('config') if config is None and par['PACKAGE'] not in ('monetdb', 'monetdb-java', 'monetdb-clients'): @@ -2950,10 +2962,15 @@ except monet_options.Error: sys.exit(1)
+ par['ALGEBRA'] = opts.get('algebra', 0) par['RECURSIVE'] = opts.get('recursive', 0) par['QUIET'] = opts.get('quiet', 0) if THISFILE == "Mtest.py": - par['IGNORE'] = opts.get('ignore', dftIGNORE) + if par['ALGEBRA'] and par['PACKAGE'] == 'monetdb-sql': + _IGNORE = '^[#%]' + else: + _IGNORE = dftIGNORE + par['IGNORE'] = opts.get('ignore', _IGNORE) par['CONTEXT'] = '-C%d' % int(opts.get('context', 1)) a = int(opts.get('accuracy', 1)) if a not in (-1,0,1,2): @@ -3132,17 +3149,17 @@ os.environ['DISTVER'] = DISTVER os.environ['VERSION'] = VERSION w = {} - for i in 'OS', 'RELEASE', 'DIST', 'VERSION', 'BITS', 'OIDS', 'FIVE', 'STATIC': + for i in 'OS', 'RELEASE', 'DIST', 'VERSION', 'BITS', 'OIDS', 'FIVE', 'STATIC', 'ALGEBRA': w[i] = re.escape(v[i]) - for i in 'BITS', 'OIDS', 'FIVE', 'STATIC': + for i in 'BITS', 'OIDS', 'FIVE', 'STATIC', 'ALGEBRA': j = 'TST_'+i env[j] = v[i] os.environ[j] = v[i] if v[i]: v[i] = '(.%s)?' % v[i] w[i] = '(\.%s)?' % w[i] - sv = '^(.(%s(%s)?|%s(%s)?))?%s%s%s%s$' % (v['OS'], v['RELEASE'], v['DIST'], v['VERSION'], v['BITS'], v['OIDS'], v['FIVE'], v['STATIC']) - sw = '^(\.(%s(%s)?|%s(%s)?))?%s%s%s%s$' % (w['OS'], w['RELEASE'], w['DIST'], w['VERSION'], w['BITS'], w['OIDS'], w['FIVE'], w['STATIC']) + sv = '^(.(%s(%s)?|%s(%s)?))?%s%s%s%s%s$' % (v['OS'], v['RELEASE'], v['DIST'], v['VERSION'], v['BITS'], v['OIDS'], v['FIVE'], v['STATIC'], v['ALGEBRA']) + sw = '^(\.(%s(%s)?|%s(%s)?))?%s%s%s%s%s$' % (w['OS'], w['RELEASE'], w['DIST'], w['VERSION'], w['BITS'], w['OIDS'], w['FIVE'], w['STATIC'], w['ALGEBRA']) r = re.compile(sw) if not r.match(par['SYSTEM']): ErrXit("System (-S) must match '"+sv+"' !") @@ -3396,6 +3413,7 @@ env['TST_OIDS'] = "" env['TST_FIVE'] = "" env['TST_STATIC'] = "" + env['TST_ALGEBRA'] = ""
is_sql = par['PACKAGE'] == 'monetdb-sql' if not is_sql: @@ -3491,7 +3509,7 @@ env['VERSION'] = os.environ['VERSION'] n = os.path.join(env['TSTTRGBASE'],TSTPREF,'.Mapprove.rc') f = open(n, 'w') - for v in 'OS', 'RELEASE', 'DIST', 'VERSION', 'TST_BITS', 'TST_OIDS', 'TST_FIVE', 'TST_STATIC': + for v in 'OS', 'RELEASE', 'DIST', 'VERSION', 'TST_BITS', 'TST_OIDS', 'TST_FIVE', 'TST_STATIC', 'TST_ALGEBRA': w = v.replace('TST_','') f.write('%s = "%s"\n' % (w, env[v])) f.close() @@ -3537,6 +3555,7 @@ product = os.path.split(env['TSTSRCBASE'])[-1] if product == 'sql': product += env['TST_FIVE'] + product += env['TST_ALGEBRA'] compiler = '' if os.name == 'nt': compiler = 'Mic' # Microsoft Visual Studio C
------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Monetdb-checkins mailing list Monetdb-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
Romulo Goncalves wrote:
Stefan Manegold wrote:
Update of /cvsroot/monetdb/MonetDB/src/testing In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv26786/MonetDB/src/testing
Modified Files: Mtest.py.in Log Message:
Added "-a / --algebra" switch to Mtest.py to enable the Algebra version of the SQL compiler (once we have a proper switch for Mserver / mclient, also Pathfinder can use this switch). Well, the option -a does not work while the option --algebra works fine.+ Yah, I was reading the wrong version of Mtest ;)
it is -g and not -a Sorry, Romulo
On Thu, Jan 31, 2008 at 04:45:09PM +0100, Romulo Goncalves wrote:
Romulo Goncalves wrote:
Stefan Manegold wrote:
Update of /cvsroot/monetdb/MonetDB/src/testing In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv26786/MonetDB/src/testing
Modified Files: Mtest.py.in Log Message:
Added "-a / --algebra" switch to Mtest.py to enable the Algebra version of the SQL compiler (once we have a proper switch for Mserver / mclient, also Pathfinder can use this switch). Well, the option -a does not work while the option --algebra works fine.+
Typo of mine. Sorry! It should read "-G / --algebra".
Yah, I was reading the wrong version of Mtest ;)
it is -g and not -a
"-G", actually ... see also `Mtest.py --help` ;-) Stefan
Sorry, Romulo
------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ 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 (2)
-
Romulo Goncalves
-
Stefan Manegold