Re: [dotNetRDF-bugs] Problems with SPARQL update

[Rob V. <rv...@do...>]

The problem was only the faulty logic in the SPARQL Update code as far as
I can see.

This is covered pretty well with testing, all the usages are in the SPARQL
engine which is covered by several hundred unit tests (523 as of writing).
 Any test that uses SPARQL will be touching this code. The SPARQL engine
is also covered by the DAWG tests i.e. the official SPARQL specification
tests which consist of another 8-900 tests

Rob

On 1/5/13 6:27 PM, "Tomasz Pluskiewicz" <tom...@gm...>
wrote:

>Ah, and indeed. When I changed my query to DELETE/INSERT the same
>problem occured for the exact same reason.
>
>This may be problematic. The Set#ContainsVariable() method is used 32
>times throughout the library and the indexer is used 141 times so it
>will be easy to break something. How well is this covered with tests?
>
>Regards,
>Tom
>
>On Sat, Jan 5, 2013 at 6:51 PM, Tomasz Pluskiewicz
><tom...@gm...> wrote:
>> Hi Rob
>>
>> I am having additional issues with running my update query.
>>
>> An equivalent select (without graph ?g {}) returns some data. However
>> when I try to create new graphs nothing happens.
>>
>> I started debugging and discovered that when evaluating the insert
>> command, all the results are skipped at InserCommand.cs.
>>
>> The problem is that VDS.RDF.Query.Algebra.Set holds variable names
>> without the question mark and in the insert command they are prefixed.
>>
>> The quick fix is to add a Substring(1) call when looking for graph
>> variablt in the set. Will similar change be required elsewhere in the
>> code?
>>
>> Regards,
>> Tom
>>
>> PS
>> When I went into debugger I and enable breaking on exceptions thrown I
>> seem to be getting a lot of Null pointer exceptions in
>> MultiDictionary. This doesn't seem to be the cause as I had though
>> initially but definitiely will have adverse effects on performance.
>
>--------------------------------------------------------------------------
>----
>Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
>MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
>with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
>MVPs and experts. SALE $99.99 this month only -- learn more at:
>http://p.sf.net/sfu/learnmore_122912
>_______________________________________________
>dotNetRDF-bugs mailing list
>dot...@li...
>https://lists.sourceforge.net/lists/listinfo/dotnetrdf-bugs