Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9929 )
Change subject: IMPALA-6549: [DOCS] Update the known issue for HDFS-12528 ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/9929/2/docs/topics/impala_known_issues.xml File docs/topics/impala_known_issues.xml: http://gerrit.cloudera.org:8080/#/c/9929/2/docs/topics/impala_known_issues.xml@450 PS2, Line 450: to be "to be" -> "are" I'd like this sentence to be clear about when short circuit reads are disabled. Something like: Use the new HDFS parameter blah to specify the amount of time short circuit reads are disabled on encountering an error. http://gerrit.cloudera.org:8080/#/c/9929/2/docs/topics/impala_known_issues.xml@453 PS2, Line 453: Nit: stray space http://gerrit.cloudera.org:8080/#/c/9929/2/docs/topics/impala_known_issues.xml@453 PS2, Line 453: 0 : is not recommended. Setting dfs.domain.socket.disable.interval.seconds to 0 is not recommended, as a non-zero interval protects the system if there is a persistent problem with short circuit reads. -- To view, visit http://gerrit.cloudera.org:8080/9929 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If51cb111a9ddc67be4a1cf42502a8a021486b7e4 Gerrit-Change-Number: 9929 Gerrit-PatchSet: 2 Gerrit-Owner: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Comment-Date: Thu, 05 Apr 2018 16:25:03 +0000 Gerrit-HasComments: Yes