asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <>
Subject Change in asterixdb[master]: Add documentation for query parameters.
Date Thu, 16 Mar 2017 19:08:06 GMT
Till Westmann has posted comments on this change.

Change subject: Add documentation for query parameters.

Patch Set 2:

File asterixdb/asterix-doc/src/main/markdown/sqlpp/

PS2, Line 76: Tuning
I think that these parameters are not part of the language, but rather a part of the system
implementing the language. So I think that we  should put this section to the end - probably
as an appendix. 

We can decide, if we want to put the SET statement into the language or into the same section
as the parameters themselves, but I would propose to keep the statement and the parameters
together for now.
File asterixdb/asterix-doc/src/main/markdown/sqlpp/

PS2, Line 4: configuration file
I think that the SQL++ documentation does contain system specific information like e.g. the
configuration file. I think that we should keep this more generic. e.g.:

    The SET statement can be used to override some cluster-wide configuration parameters for
a specific request:

PS2, Line 8: Since each parameter described as follows
           : is a fully qualified name, we have to use a backtick (\`\`) to make it a delimited
As parameter identifier are qualified names (containing a '.') they have to be escaped using
backticks (\`\`).

PS2, Line 10: According to the syntax, the value part of each parameter has to always be a
I think that we don't need this sentence, as we already say that it's a STRING_LITERAL further

PS2, Line 15: multiple
s/multiple/multiple cores on multiple/

PS2, Line 15: Since the
s/Since the/The/

PS2, Line 15: individual query
s/individual query/request/

PS2, Line 16: , a
s/, a/. A/

PS2, Line 16: query

PS2, Line 16: ,
            : based on on her/his own need,
s/, based on on her/his own need,//

PS2, Line 33: memory footprint
s/memory footprint/fixed memory/

PS2, Line 33: even

PS2, Line 34: much 

PS2, Line 34: given

PS2, Line 35: Using the `SET` statement, a
s/Using the `SET` statement, a/A/

PS2, Line 36:  listed as follows
a/ listed as follows//

PS2, Line 52:   
Just one space?

PS2, Line 60:   
Just one space?

PS2, Line 68:   
Just one space?

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I80dcd668bea3b2b3fff0c0778548ffad63505d99
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Michael Blow <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-HasComments: Yes

View raw message