Difference between revisions of "To PR"

From SPARQL Working Group
Jump to: navigation, search
(Query (LC))
(PR Reviews)
Line 74: Line 74:
 
=== PR Reviews ===
 
=== PR Reviews ===
  
* Carlos (ACTION-643)
+
* [http://lists.w3.org/Archives/Public/public-rdf-dawg/2012JulSep/0065.html Carlos] ([ACTION-643)
 
* Olivier (ACTION-644) - DONE
 
* Olivier (ACTION-644) - DONE
  

Revision as of 14:53, 4 September 2012

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