Bugs item #1714360, was opened at 2007-05-07 16:31
Message generated for change (Comment added) made by sjoerd
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1714360&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/runtime
Group: Pathfinder CVS Head
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Jan Rittinger (tsheyar)
Assigned to: Peter Boncz (boncz)
Summary: PF/runtime: find_qn_bulk() adds new qnames twice

Initial Comment:
If I compile and run the query 
BugTracker/nested_element_construction.SF-1712578.xq with the algebra version 
(pf -A) I get a merged_union error which boils down to duplicate entries in the 
qname relations. The reason are the following two lines from find_qn_bulk() in 
pf_support.mx which both modify the same relation:

1.188        (stmane   10-Jan-07):                         
qn_prefix_uri_loc.append(_pref_uri_loc, true);

1.195        (sjoerd   20-Feb-07):       ws.fetch(QN_PREFIX_URI_LOC + 
UPDATE).find(cont).insert(_pref_uri_loc);


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

>Comment By: Sjoerd Mullender (sjoerd)
Date: 2007-07-20 17:36

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

If I run the test
tests/BugTracker/Tests/nested_element_construction.SF-1712578_alg, I get no
differences.  So what is the problem then?
If the problem is what you describe in your comment about add_qnames
behaving differently than [add_qname], then that could perhaps be a
different bug.  I don't immediately see how that relates to find_qn_bulk,
which is what the bug was originally about.
In other words, if this is a new bug, please use a new bug report, if it
is the same bug but in a different guise, please elaborate, and if there is
no bug, then please close this report.

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

Comment By: Jan Rittinger (tsheyar)
Date: 2007-06-26 11:12

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

This bug seems to be open again.

module ("pathfinder");
var ws := ws_create ();
var a := bat
(void,str).seqbase([EMAIL 
PROTECTED]).append("foo").append("foo").append("foo").append("bar").append("bar").append("bar");
var b := add_qnames(a.project (""), a.project (""), a, ws);
print(ws.fetch(QN_LOC).fetch(0));
#-----------------#
# h     t         # name
# oid   str       # type
#-----------------#
[ [EMAIL PROTECTED],    "foo"   ]
[ [EMAIL PROTECTED],    "bar"   ]
[ [EMAIL PROTECTED],    "foo"   ]
[ [EMAIL PROTECTED],    "bar"   ]

Whereas the following snippet does the correct thing:
var b := [add_qname](a.project (""), a.project (""), a, CONST ws);
#-----------------#
# h     t         # name
# void  str       # type
#-----------------#
[ [EMAIL PROTECTED],    "foo"   ]
[ [EMAIL PROTECTED],    "bar"   ]


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

Comment By: Stefan Manegold (stmane)
Date: 2007-05-18 07:26

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

finishing Peter'\''s fix of find_qn_bulk():

the new code used "if (update) ...",
while the function'\''s signature read "... bit add ...";
this caused about 20 tests to fail with
"!ERROR: unknown variable '\''update'\''."

fixed by renaming the parameter "add" to "update"
(the latter indeed seems more intuitive/descriptive than the former).

this makes the ~20 failing tests work, again.

test added in
pathfinder/tests/BugTracker/Tests/nested_element_construction.SF-1712578_alg.*


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

Comment By: Peter Boncz (boncz)
Date: 2007-05-17 23:50

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

fixed

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

Comment By: Stefan Manegold (stmane)
Date: 2007-05-07 17:43

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

Just as a quick info:

===================================================================
2007/01/10 - stmane: runtime/pf_support.mx,1.188
propagated changes of Friday Jan 05 2007 - Wednesday Jan 10 2007
from the XQuery_0-14 branch to the development trunk
===================================================================

===================================================================
2007/02/20 - sjoerd: runtime/pf_support.mx,1.195
propagated changes of Sunday Feb 11 2007 - Tuesday Feb 20 2007
from the XQuery_0-16 branch to the development trunk
===================================================================


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

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

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs

Reply via email to