The Open Anzo Project

Semantic Application Middleware

Ticket #295 (closed defect: fixed)

Opened 9 months ago

Last modified 8 months ago

poor handling of nulls in JSONRdfWriter and respective anzo.js reader

Reported by: jpbetz Assigned to: jordi
Priority: major Milestone: 3.0 milestone1
Component: openanzo Version: 3.0
Keywords: bnodes Cc: jordi rmeschian

Description (Last modified by jordi)

The lax handling of nulls in the JSONRdfWriter and anzo.js reader (and presumably the writer and reader going the other direction as well) can make debugging difficult.

This was particularly true for a recent bnode issue where BlankNodes created with a different BlankNodeManager than used by the JSONRdfWriter were serialized to null because the JSONRdfWriter was unable to find a label for the BlankNodes since it's manager had not created them.

This is how the null blank node looks on the wire:

... subject":{"objectType":"bnode","value": null } ...
...
... object":{"objectType":"bnode","value": null } ...

On the anzo.js side, the null valued blank node was not rejected (no exception or error was thrown), but it was not correlated with the other uses of the same blank node within the graph either.

The result is that in the constructed anzo.js graph, a single blank node appearing in a number of statements was treated as numerous, unrelated, blank nodes.

Change History

04/23/08 22:21:54 changed by jpbetz

(In [2157]) re #295

Throw exception rather than serialize null blank node label. Also add unrelated parse convenience call to Column class.

04/24/08 12:50:33 changed by jpbetz

(In [2162]) re #295

Use common blank node manager in ldap solution generator to avoid issue described in #295.

04/24/08 13:34:02 changed by jordi

  • description changed.

05/02/08 17:25:26 changed by jpbetz

  • owner changed from jpbetz to jordi.

Jordi,

Would you review and close?

-Joe

05/14/08 18:35:48 changed by jordi

  • status changed from new to closed.
  • resolution set to fixed.

Looks good.

Copyright © 2007 - 2008 OpenAnzo.org