Abhishek Rawat has posted comments on this change. ( http://gerrit.cloudera.org:8080/23852 )
Change subject: IMPALA-14661: Optimize admissiond memory usage by compressing exec requests ...................................................................... Patch Set 2: (4 comments) Some initial thoughts. http://gerrit.cloudera.org:8080/#/c/23852/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23852/2//COMMIT_MSG@13 PS2, Line 13: compressing TQueryExecRequest Is it possible to do this based on a size threshold of TQueryExecRequest? This should help avoid the compression/decompression cost for small/simple queries. http://gerrit.cloudera.org:8080/#/c/23852/2//COMMIT_MSG@17 PS2, Line 17: starting nit: startup http://gerrit.cloudera.org:8080/#/c/23852/2//COMMIT_MSG@25 PS2, Line 25: reduce : memory Should this be: avoid decompressing multiple times? http://gerrit.cloudera.org:8080/#/c/23852/2//COMMIT_MSG@26 PS2, Line 26: Requests are : initially decompressed at submission Maybe we should consider only doing this if admission queue is empty or admissiond's memory is not close to its memory limit. -- To view, visit http://gerrit.cloudera.org:8080/23852 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5a676d1a806451cbf84b0a3f8a706d7c6655e12d Gerrit-Change-Number: 23852 Gerrit-PatchSet: 2 Gerrit-Owner: Yida Wu <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Tue, 13 Jan 2026 01:10:59 +0000 Gerrit-HasComments: Yes
