Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20623 )

Change subject: IMPALA-12516: Set HDFS limit based on memlock
......................................................................


Patch Set 8:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/20623/7/testdata/cluster/admin
File testdata/cluster/admin:

http://gerrit.cloudera.org:8080/#/c/20623/7/testdata/cluster/admin@250
PS7, Line 250:       ULIMIT_LOCKED_MEM=65536
> There's an argument it should be 1/3rd of this, since we're running 3 datan
I think this is misplaced thinking. I'd read something about systemd having a 
hard limit of 64 MB, but that was a bug and probably still per-process.


http://gerrit.cloudera.org:8080/#/c/20623/7/testdata/cluster/admin@253
PS7, Line 253:     let DATANODE_LOCKED_MEM=$ULIMIT_LOCKED_MEM*333
> This should probably match what we're setting in bootstrap_system.sh.
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7722ddd0c7fbd9bbd1979503952b7522b808194a
Gerrit-Change-Number: 20623
Gerrit-PatchSet: 8
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: David Rorke <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Sat, 04 Nov 2023 00:04:53 +0000
Gerrit-HasComments: Yes

Reply via email to