impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Russell (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3200: [DOCS] Document user-facing aspects of new buffer pool
Date Thu, 05 Oct 2017 00:29:35 GMT
John Russell has posted comments on this change. ( )

Change subject: IMPALA-3200: [DOCS] Document user-facing aspects of new buffer pool

Patch Set 4:

File docs/topics/impala_buffer_pool_limit.xml:
PS4, Line 53: MiB
> MB. I may have mixed up MB / KB and MiB / KiB in a few places throughout. M
File docs/topics/impala_max_row_size.xml:
PS4, Line 68: <codeblock><![CDATA[
> This example is really helpful! Hopefully this will head off some support c
Done. TY. That was also a good catch by you of the outdated spilling details.
PS4, Line 82: 500 KB string and a 30 KB
> These are actually KiB. I didn't mean to actually cut it so close (530,000 
PS4, Line 95: KiB
> Agh, KiB again where it should be KB.
PS4, Line 109: Increase the max_row_size query option (currently 512.00 KB) to process larger
> Wrap these very long error message lines.

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I49323f8ffbff3e195058e88762eedbb1fcb1bc0e
Gerrit-Change-Number: 8003
Gerrit-PatchSet: 4
Gerrit-Owner: John Russell <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: John Russell <>
Gerrit-Reviewer: Mostafa Mokhtar <>
Gerrit-Reviewer: Tim Armstrong <>
Gerrit-Comment-Date: Thu, 05 Oct 2017 00:29:35 +0000
Gerrit-HasComments: Yes

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message