Alex Behm has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10109 )

Change subject: IMPALA-6886: [DOCS] Removed impala_cluster_sizing.xml
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10109/1/docs/impala.ditamap
File docs/impala.ditamap:

http://gerrit.cloudera.org:8080/#/c/10109/1/docs/impala.ditamap@50
PS1, Line 50:     <!-- Removed per Alan Choi's request on 4/18/2018 <topicref 
href="topics/impala_cluster_sizing.xml"/> -->
> It might come back some day
I'm not a big fan of leaving speculative bread crumbs since nobody knows what 
the future will bring. Chances are we're never going to need this again, and if 
we do, it's easy to find with git </rant>. Let's not dwell and move on with 
this.



--
To view, visit http://gerrit.cloudera.org:8080/10109
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I03d605d33ed6ced809074b1fc96def30ad0887fd
Gerrit-Change-Number: 10109
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Alan Choi <a...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Comment-Date: Thu, 19 Apr 2018 04:34:16 +0000
Gerrit-HasComments: Yes

Reply via email to