You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Sanjana Malhotra (Code Review)" <ge...@cloudera.org> on 2026/10/21 16:01:53 UTC

[Impala-ASF-CR] IMPALA-13839: [DOCS]MAX FS WRITERS query option documentation

Sanjana Malhotra has posted comments on this change. ( http://gerrit.cloudera.org:8080/25119 )

Change subject: IMPALA-13839: [DOCS]MAX_FS_WRITERS query option documentation
......................................................................


Patch Set 2:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/25119/2/docs/topics/impala_max_fs_writers.xml
File docs/topics/impala_max_fs_writers.xml:

http://gerrit.cloudera.org:8080/#/c/25119/2/docs/topics/impala_max_fs_writers.xml@40
PS2, Line 40: file
            :       system
> Kudu is not a "file system". We can just remove "file system" here.
Done


http://gerrit.cloudera.org:8080/#/c/25119/2/docs/topics/impala_max_fs_writers.xml@51
PS2, Line 51: <keyword keyref="impala40_full"/>
> I don't see this is displayed:
Done


http://gerrit.cloudera.org:8080/#/c/25119/2/docs/topics/impala_max_fs_writers.xml@62
PS2, Line 62: between scan operations and
            :       table sinks
> Scan is just an example. The table sink is in the root fragment. Maybe say:
Done


http://gerrit.cloudera.org:8080/#/c/25119/2/docs/topics/impala_max_fs_writers.xml@73
PS2, Line 73:       A mismatch between cluster composition during query planning and execution can result in
> This is unclear to me. What does "cluster composition" mean here? Looking i
Done


http://gerrit.cloudera.org:8080/#/c/25119/2/docs/topics/impala_max_fs_writers.xml@74
PS2, Line 74: The scheduler logs warnings when this
            :       occurs.
> Can we add an example for the warning logs? I think it's this log:
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd39039bfa46f233cbf94e9ce7d967eed75c27fb
Gerrit-Change-Number: 25119
Gerrit-PatchSet: 2
Gerrit-Owner: Sanjana Malhotra <ma...@gmail.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <hu...@gmail.com>
Gerrit-Reviewer: Sanjana Malhotra <ma...@gmail.com>
Gerrit-Comment-Date: Wed, 21 Oct 2026 16:01:53 +0000
Gerrit-HasComments: Yes