To PR

From SPARQL Working Group
Revision as of 14:53, 4 September 2012 by Apollere2 (Talk | contribs)

Jump to: navigation, search

This page is for the SPARQL WG to record issues to be resolved and tasks to be completed before documents are ready for going to Proposed Recommendation.

Overall PR/final editing requirements

  • Do we still need to align style for "must, must not, should, should not, may and recommended" RFC2119 terms across documents? cf. http://lists.w3.org/Archives/Public/public-rdf-dawg/2011AprJun/0050.html
  • Remove all "red boxes"
  • set review period as soon as pub date is fixed.
  • mark as PR
  • Do we have to remove all "AT RISK" marks? (Axel: I understand that this can only remain during CR, so if we skip CR, I guess we have to make a decision)
  • Any process-wise necessary Template text for PR?
  • List W3C members we can ask for endorsement during PR review phase
  • contact RDF WG for official reviews before we go to PR (with LC publication of query?)

Test Suite

  • All test cases approved and passed by two independend implementations
  • Test suite README.html document in place and ready

Query (LC)

3LC: 24 July 2012 until 21 August 2012.

Document: Editors working draft

LC Reviews

Comments

  • Andy: 5 open comments; 3 on bindings, 1 on prefixed names, 1 on VALUES word [DONE]

Issues

  • Formally accept VALUES changes.
  • Check scope text (Axel)
  • Algebra generation for BIND needs sorting out [DONE]
  • Nested Aggregates
    • Grammar note saying "it's illegal to nest aggregate in expressions"
    •  ?? a note at the end of aggregates ("11.5 Aggregate Example (with errors)") for SELECT Expressions and aggregates assigned to vars (Steve)
  • Should IRIREF include UCHAR? (no - \u does earlier in SPARQL for compatibility with SPARQL 1.0)
  • Populate the function namespace page ACTION-389.

"At Risk" Boxes

Editorial Tasks

  • Axel: in-scope definition fix
  • Andy: fix / cleanup references
  • Andy: remove at-risk box in grammar (requires group decision)
  • Steve: 18.5.1: "solutions containing error values are removed at projection time." -- does not reflect that it's bindings that are errors leave the variable unbound but the solution (i.e. row) remains.


  • SAMPLE returns RDFTerm not literal [DONE]
  • Steve: Define MIN, MAX, SAMPLE of empty set (to be error) [DONE]
  • Andy: STRBEFORE/STRAFTER changes related to error condition [DONE]

Implementation Reports/Tests (for PR only)

Update (PR)

PR Reviews

  • Carlos ([ACTION-643)
  • Olivier (ACTION-644) - DONE

Comments

Issues

Editorial Tasks

Implementation Reports/Tests

Protocol (PR)

FINISHED (i.e. no more full review needed - No changes since last call), except the following coments and minor issues:

Comments

  • Lee to draft response to RC-1, RC-2, MLR-1

Editorial Tasks

  • Fix 2.2.2 to note that UTF-8 is the charset there

Implementation Reports/Tests

Service Description (PR)

FINISHED (i.e. no more full review needed - No changes since last call), except the following coments and minor issues:


Comments

  • All comments have been responded to; pinged 3 commenters where we don't have final acknowledgement

Implementation Reports/Tests

  • Validator tests basic SD conformance
  • ACTION-632
  • ACTION-633
  • ACTION-634
  • RDF::Query passes the tests; Chimezie has basic implementation & will try out; Steve has an out of date impl that might be updated

Graph Store HTTP Protocol (PR)

FINISHED (i.e. no more full review needed - No significant changes since last call), except the following coments and minor issues:


PR Reviews

  • No changes made since LC publication, except removed refs to REST
  • Review pending: Axel (ACTION-645) this review is not critical path (since there are no significant changes in the doc, decided in Telco 2012-07-03 that no more full review was needed)

Comments

  • JL-3, JL-4


Implementation Reports/Tests

  • Test case coverage?
  • Are there reports of two independent implementations passing all related test cases?

Entailment (PR)

FINISHED (i.e. no more full review needed - No significant changes since last call), except the following coments and minor issues:

Comments

  • No, just some ACKs missing from Michael Schneider

Implementation Reports/Tests

  • Test case coverage still weak
  • No reports yet, ACTION-637 to ask candidate implementors

Federated Query (PR)

FINISHED (i.e. no more full review needed - No significant changes since last call), except the following coments and minor issues:


Comments

  • Open comment from Peter Waher PW-1. Draft at response, working on it

Editorial Tasks

  • Any Open Editorial tasks?

Change BINDINGS for VALUES

Implementation Reports/Tests

  • Test case coverage? Missing tests, ACTION-520
  • Are there reports of two independent implementations passing all related test cases?

JSON Results (PR)

FINISHED (i.e. no more full review needed - No significant changes since last call), except the following coments and minor issues:

Document: Editors working draft

no changes since last version, no reviews needed.


Implementation Reports/Tests

  • Test case coverage?
  • Are there reports of two independent implementations passing all related test cases?

CSV-TSV Results Format (PR)

FINISHED (i.e. no more full review needed - No significant changes since last call), except the following coments and minor issues:

Document: Editors working draft

Implementation Reports/Tests

  • Test case coverage?
  • Are there reports of two independent implementations passing all related test cases?

Reviews

  • chime (ACTION-647) (non critical path, since no changes since LC and no critical comments)

Overview document

FINISHED (i.e. no more full review needed - No significant changes since last call), except the following coments and minor issues:


Editorial Tasks