To PR

From SPARQL Working Group
Revision as of 14:45, 3 July 2012 by Pgearon (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)

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