Ali Alsuliman 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 2:

(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 diffe
Yeah, I didn't want to change the existing one, but I changed it now. In the 
grammar, it's <TYPE> of course. That "?" should be removed also I believe, 
correct (I guess by accident)?


PS1, Line 266: 
> WS
Done


PS1, Line 267: 
             : 
> Use the same formatting/indentation as for the other examples (instead of <
Done


-- 
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: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman <ali.al.solai...@gmail.com>
Gerrit-Reviewer: Ali Alsuliman <ali.al.solai...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Carey <dtab...@gmail.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: Yes

Reply via email to