Matthew Jacobs has posted comments on this change. Change subject: [DOCS] Explain Boost setting needed for 96-bit timestamps ......................................................................
Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7983/2/docs/shared/impala_common.xml File docs/shared/impala_common.xml: PS2, Line 848: our who is 'our' here? PS2, Line 850: , : which matches the representation for <codeph>TIMESTAMP</codeph> in Impala , and is required in order to use TIMESTAMP. Line 855: <codeblock> it might be good to just say that you should probably just define this variable always, unless you have a really good reason not to -- To view, visit http://gerrit.cloudera.org:8080/7983 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4b67cd7762f682c3a054e0d9641080aa51801c83 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: John Russell <jruss...@cloudera.com> Gerrit-Reviewer: John Russell <jruss...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: Taras Bobrovytsky <tbobrovyt...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-HasComments: Yes