Code review comment for lp:~niedbalski/charm-helpers/lp-1461669

Revision history for this message
Mario Splivalo (mariosplivalo) wrote :

LGTM too. The default is 50%, although, maybe it would be best to have default of 256M, unless available memory is less than that (then default to 20% of RAM, or something like it)?

review: Approve

« Back to merge proposal