Re: [dotNetRDF-bugs] VirtuosoStorage bug report

[Robert V. <rv...@vd...>]

Hi Alexander

 

Thanks for the bug report, I have applied the first fix as-is and the second
fix I have done by just changing the branch to an else instead, I’ve also
made some other changes to improve the Virtuoso support based on feedback
from my contact at OpenLink

 

Can you confirm for me that the fixes work for your queries?  Revision 575
or higher has the fixes

 

Also can you send me the test queries (and data if applicable) that produced
the bug so I can add some additional test cases to my test suite

 

Thanks,

Rob Vesse

 

  _____  

From: Aleksandr A. Zaripov [mailto:za...@tp...] 
Sent: 05 February 2010 17:26
To: dot...@li...
Subject: SPAM-LOW: [dotNetRDF-bugs] VirtuosoStorage bug report

 

Hello.
During my work with dotNetRDF for my science reseaches about Semantic Web I
found some bugs in Query method of the VirtuosoManager class (I check out
the revision #572).

The first bug is that the code in the lines 677-683 does not return
anything. I guess that should add lines like

 rset.AddResult(res);
 finalResult = rset;

below line 683.

The second bug is that if SPARQL query returns 1 row and more than 1 column
it will be ignored. I guess, that this queries should be processed by the
code in lines 692-717 (like queries, which returns multiple rows). So, I
offer to change the line 690 to

 else if (results.Rows.Count > 1 || results.Columns.Count > 1)

or may be other solution for this place.

Diff file is attached to this mail.

Best regards, Alexander A. Zaripov, Tomsk Polytechnic University
za...@tp...



No virus found in this incoming message.
Checked by AVG - www.avg.com
Version: 8.5.435 / Virus Database: 271.1.1/2671 - Release Date: 02/06/10
07:35:00