ADQL-2_0-Err-1 < IVOA

IVOA Web>IvoaDAL>ADQL>ADQL-2_0-Errata>ADQL-2_0-Err-1 (2018-02-23, MarcoMolinaro) (raw view)

---+ ADQL-2.0 Erratum 1: White space clarification

Author: Dave Morris

Date last changed: 2018-02-23

Date accepted: 2018-02-22

---++ Rationale

The BNF grammar for the [[http://www.ivoa.net/Documents/cover/ADQL-20081030.html][ADQL-2.0]] language is listed in Appendix A of the recommendation. In this grammar a =separator= rule is present, with the following definition:
<verbatim>
<separator> ::= { <comment> | <space> | <newline> }...
</verbatim>

However this nonterminal token is only referenced in the rule for the =character_string_literal=, i.e.
<verbatim>
<character_string_literal> ::=
  <quote> [ <character_representation>... ] <quote>
  [ { <separator>... <quote> 
  [ <character_representation>... ] <quote> }... ]
</verbatim>

It is uncontroversial that the intent is to allow comments and white-space wherever SQL-92 allows them. The !ADQL standard however says differently, and there should be a clarification. Between two alternatives, adding a clarifying subsection to the !ADQL-2.0 standard or removing the =separator=.

---++ Erratum Content

In section 2 of the ADQL 2.0 Recommendation, add a section 2.1.4 with the following content:

<verbatim>
1.1.4 Whitespace and Comments     

The rules on where whitespace is allowed and required are as in SQL92;
essentially, any <token> may be followed by a <separator>.
</verbatim>

---++ Impact Assessment

The change introduced by this erratum has no other impact on the current ADQL grammar and language apart from what already described in the previous sections of this note. Current implementations and parsers of ADQL query strings shouldn't be affected at all by this change. Moreover, current BNF grammar in !ADQL-2.0 needs changes to be machine readable and validated and one of the issues in BNF is exactly how to deal with optional or required white spaces.

---++ Note

This Erratum was previously part of the [[%ATTACHURL%/ADQL2Err1.pdf][ADQL2Err1 Errata Note (PDF former version)]].

Erratum approved on 2018-02-22 by TCG, but last changes made as action deriving from that TCG tconf: removing unstable SVN reference and added formal author. -- IVOA.MarcoMolinaro - 2018-02-23

<!--
      * Set ALLOWTOPICRENAME = IVOA.TWikiAdminGroup
-->

Attachments

Topic attachments
I	Attachment	History	Action	Size	Date	Who	Comment
pdf	ADQL2Err1.pdf	r1	manage	192.2 K	2018-02-23 - 09:26	MarcoMolinaro	ADQL-2.0 Erratum 1, former PDF version

Topic revision: r5 - 2018-02-23 - MarcoMolinaro

IVOA

Log in or Register

IVOA.net
Wiki Home
WebChanges
WebTopicList
WebStatistics

Twiki Meta & Help
IVOA
Know
Main
Sandbox
TWiki

TWiki intro
TWiki tutorial
User registration
Notify me

Working Groups

Interest Groups

Time Domain

Committees

Stds&Procs

www.ivoa.net
Documents
Events
Members
XML Schema