Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/14627 )
Change subject: IMPALA-9085: [DOCS] Refactored impala_s3.xml ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/14627/3/docs/topics/impala_s3.xml File docs/topics/impala_s3.xml: http://gerrit.cloudera.org:8080/#/c/14627/3/docs/topics/impala_s3.xml@a406 PS3, Line 406: > why delete this example? I don't see the example adding values. It is repeating what's already said or describing things very straightforard. Let me know if you feel this whole section should be added back in. -- To view, visit http://gerrit.cloudera.org:8080/14627 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib274968a0412b4b8757f31ab674d4b82311de70a Gerrit-Change-Number: 14627 Gerrit-PatchSet: 3 Gerrit-Owner: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com> Gerrit-Comment-Date: Fri, 08 Nov 2019 19:31:40 +0000 Gerrit-HasComments: Yes