Bugs item #1981735, was opened at 2008-06-02 08:43
Message generated for change (Comment added) made by sjoerd
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1981735&group_id=56967

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: PF/tests
Group: Pathfinder "stable"
Status: Closed
Resolution: Fixed
Priority: 2
Private: No
Submitted By: Stefan Manegold (stmane)
Assigned to: Peter Boncz (boncz)
Summary: PF/alg: some error messages differ from PF/mps

Initial Comment:
With the Algebra back-end, some error messages seem to differ from the 
milprint_summer back-end.

In case this is intended, we should (must) approved the Algebra-specific output
(as I already did with
tests/BugDay_2005-12-19_0.9.3/Tests/tag_name_validation.SF-1221984.stable.err{,.Algebra}
tests/BugDay_2005-12-19_0.9.3/Tests/zero-or-one_merged-union.SF-1221336.stable.err{,.Algebra}
).
Otherwise, we should (must) fix the error messages.

Examples:
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_i18n/bad-001.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/matches02.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/matches01.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/attr_err.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/div.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/cast2.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_W3C_use_cases_XQUF_R/Q4.out.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_BugsViaSourgeforce/invisible_error_messages.SF-1409122.err.00.html


----------------------------------------------------------------------

>Comment By: Sjoerd Mullender (sjoerd)
Date: 2009-11-17 10:11

Message:
I now have a patch ready for the problem I identified in my last comment. 
It's too late for the release build I'm doing now, so if this patch proofs
to be good, it'll come in the next release.
The error message I now get contains the complete invalid character, not
just the first byte of its UTF-8 encoding.

----------------------------------------------------------------------

Comment By: Sjoerd Mullender (sjoerd)
Date: 2009-11-17 09:31

Message:
Actually, I think this does show a (small) problem in the compiler: when
confronted with a multi-byte character that it doesn't expect, it only
prints the first byte.  The compiler should treat the document as a
sequence of Unicode code point, not as a sequence of bytes.

----------------------------------------------------------------------

Comment By: Stefan Manegold (stmane)
Date: 2009-11-16 23:38

Message:
Thanks.
I approved the Linux-specific error message.


----------------------------------------------------------------------

Comment By: Peter Boncz (boncz)
Date: 2009-11-16 23:23

Message:
Stefan, I'd say this seems to have to do with different locales on
different test machines.



----------------------------------------------------------------------

Comment By: Stefan Manegold (stmane)
Date: 2009-11-14 22:37

Message:
The issues is not gone, yet.
The question is still, why the same query triggers different error
messages on different systems, and/or whether this matters or whether we
just accept the differences as given.

We seems to get either
"
ERROR = !parse error: syntax error, unexpected invalid_character on line
5, column 19 (next token is `�')
        !parse error: XQuery parsing failed
"
or simply
"
ERROR = !        !parse error: XQuery parsing failed
"

cf.
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTests103/GNU.64.64.d.1-Fedora10/tests_i18n/bad-001.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsg103/GNU.64.64.d.1-Fedora10/tests_i18n/bad-001.err.00.html


----------------------------------------------------------------------

Comment By: Peter Boncz (boncz)
Date: 2009-11-12 10:37

Message:
after my latest pass over the web, such issues should be gone

----------------------------------------------------------------------

Comment By: Stefan Manegold (stmane)
Date: 2009-02-14 19:20

Message:
for this one, the error message appears to be identical (now) with ALG &
MPS, but differs from what it used to be, giving less information than
before:

ALG:
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTests103/GNU.32.32.d.1-Debian4.0/tests_i18n/bad-001.err.00.html
MPS:
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsg103/GNU.32.32.d.1-Debian4.0/tests_i18n/bad-001.err.00.html


----------------------------------------------------------------------

Comment By: Stefan Manegold (stmane)
Date: 2009-02-14 18:41

Message:
For the records:

Only one seems to remain:

http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTests103/GNU.64.64.d.1-Fedora8/tests_XQuery/div.err.00.html


----------------------------------------------------------------------

Comment By: Lefteris Sidirourgos (lsidir)
Date: 2008-06-12 14:34

Message:
Logged In: YES 
user_id=1856546
Originator: NO

Lower the priority since the remaining two different error messages are
not important

----------------------------------------------------------------------

Comment By: Lefteris Sidirourgos (lsidir)
Date: 2008-06-09 15:11

Message:
Logged In: YES 
user_id=1856546
Originator: NO

The following tests are now producing the correct error msg:
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/matches02.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/matches01.err.00.html

----------------------------------------------------------------------

Comment By: Stefan Manegold (stmane)
Date: 2008-06-03 13:43

Message:
Logged In: YES 
user_id=572415
Originator: YES

Still differing:
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_i18n/bad-001.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/matches02.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/matches01.err.00.html
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTestsG103/GNU.64.64.d-Fedora8/tests_XQuery/div.err.00.html


----------------------------------------------------------------------

Comment By: Stefan Manegold (stmane)
Date: 2008-06-03 11:33

Message:
Logged In: YES 
user_id=572415
Originator: YES

With the updated error message of invisible_error_messages.SF-
1409122.stable.err, the MPS test now fails

http://monetdb.cwi.nl/testing/projects/monetdb/Stable/pathfinder/.mTests103/GNU.64.32.d-Fedora8/tests_BugsViaSourgeforce/invisible_error_messages.SF-1409122.err.00.html


----------------------------------------------------------------------

Comment By: Jan Rittinger (tsheyar)
Date: 2008-06-02 15:57

Message:
Logged In: YES 
user_id=993208
Originator: NO

Fixed attr_err.err, cast2.err, Q4.out, and invisible_error_messages.SF-
1409122.err.

Still remaining:
bad-001.err (could it be that mclient has problems with the characters?)
matches (CATCH is missing in the code generation)
div (check for denominator in milgen is missing)

Re-assigned to Lefteris as he can fix the MIL specific problems (and he
started fn:matches).

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1981735&group_id=56967

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs

Reply via email to