Difference between revisions of "To PR"

From SPARQL Working Group
Jump to: navigation, search
m (PR Reviews)
m (Editorial Tasks)
Line 88: Line 88:
 
* Any Open Editorial tasks? (email pending)
 
* Any Open Editorial tasks? (email pending)
 
* check again changes as per [http://www.w3.org/2009/sparql/track/actions/585 ACTION-585]
 
* check again changes as per [http://www.w3.org/2009/sparql/track/actions/585 ACTION-585]
 +
* Make editorial changes as per [http://www.w3.org/2009/sparql/track/actions/644 ACTION-644]
  
 
=== Implementation Reports/Tests ===
 
=== Implementation Reports/Tests ===

Revision as of 14:45, 3 July 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)

Document: Editors working draft

LC Reviews

Comments

  • Andy: 4 open comments; 3 on bindings, 1 on prefixed names

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: STRBEFORE/STRAFTER changes related to error condition [DONE]
  • Andy: fix / cleanup references
  • Andy: remove at-risk box in grammar (requires group decision)
  • SAMPLE returns RDFTerm not literal [DONE]
  • Steve: Define MIN, MAX, SAMPLE of empty set (to be error)
  • Steve: 18.5.1: "solutions containing error values are removed at projection time." -- does not reflect that it's bindins that are errors leave the varaibale unbound but the solution (i.e. row) remains.

Implementation Reports/Tests (for PR only)

Update (PR)

PR Reviews

  • Two reviews assigned? Completed? (email pending)
  • Carlos (ACTION-643)

Comments

Issues

Editorial Tasks

  • Any Open Editorial tasks? (email pending)
  • check again changes as per ACTION-585
  • Make editorial changes as per ACTION-644

Implementation Reports/Tests

Protocol (PR)

PR Reviews

  • No changes since last call; no reviewers needed

Comments

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

Issues

  • No open issues

Editorial Tasks

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

Implementation Reports/Tests

Service Description (PR)

PR Reviews

  • No significant changes; no reviews needed

Comments

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

Issues

  • Possible issue regarding sd: sd:DereferencesURIs definition (brought on by David Booth's recent comment). Will send email to the list about this and discuss options.

Editorial Tasks

  • None

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)

PR Reviews

  • No changes made since LC publication.
  • removed refs to REST

Comments

  • MSO-1 - DONE
  • JL-3

Issues

  • No issues

REVIEWS

  • Axel (ACTION-645)

Editorial Tasks

  • Clarify use of HTTP status codes per MSO-1

Implementation Reports/Tests

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

Entailment (PR)

PR Reviews

  • No reviews assigned because there were no changes

Comments

  • No, just some ACKs missing from Michael Schneider

Issues

  • No open issues

Editorial Tasks

  • No open editorial tasks

Implementation Reports/Tests

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

Federated Query (PR)

PR Reviews

  • No changes since last call; no reviewers needed

Comments

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

Issues

  • No open issues

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)

Document: Editors working draft

no changes since last version, no reviews needed.

PR Reviews

  • Two reviews assigned? Completed?

Comments

  • Any Open comments?

Issues

  • Any Open related Issues?

Editorial Tasks

  • Any Open Editorial tasks?

Implementation Reports/Tests

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

CSV-TSV Results Format (PR)

Document: Editors working draft

PR Reviews

  • no changes, no reviews needed.

Comments

  • Any Open comments?

Issues

  • Any Open related Issues?

Editorial Tasks

  • Any Open Editorial tasks?

Implementation Reports/Tests

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

Reviews

  • chime (ACTION-647)

Overview document

PR Reviews

  • Two reviews assigned? Completed?

Comments

  • Any Open comments?

Editorial Tasks