You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Till Westmann (Code Review)" <do...@asterixdb.incubator.apache.org> on 2018/11/05 22:22:26 UTC

Change in asterixdb[master]: [ASTERIXDB-2463][DOC] add docs for primary index & parallel ...

Till Westmann has posted comments on this change.

Change subject: [ASTERIXDB-2463][DOC] add docs for primary index & parallel sort parameters - user model changes: no - storage format changes: no - interface changes: no
......................................................................


Patch Set 1:

(3 comments)

https://asterix-gerrit.ics.uci.edu/#/c/3015/1/asterixdb/asterix-doc/src/main/markdown/sqlpp/5_ddl_dataset_index.md
File asterixdb/asterix-doc/src/main/markdown/sqlpp/5_ddl_dataset_index.md:

PS1, Line 196: "type"
Use <TYPE> instead (in the grammar and the docs)? Or does that have a different meaning?


PS1, Line 266:     
WS


PS1, Line 267: <br />
             : `SELECT COUNT(*) FROM GleambookMessages;`
Use the same formatting/indentation as for the other examples (instead of <br/> and backticks?


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/3015
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaad725f7104da7f70b1064ffda90e5397b3094ec
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dt...@gmail.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes