[kudu-CR] log block manager: use sparse hash map for block map

2017-03-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: log_block_manager: use sparse_hash_map for block map .. log_block_manager: use sparse_hash_map for block map Based on my testing, this reduces the memory

[kudu-CR] log block manager: use sparse hash map for block map

2017-03-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log_block_manager: use sparse_hash_map for block map .. Patch Set 4: (1 comment) looks like using the GNUC macro wasn't useful because clang can still be using

[kudu-CR] log block manager: use sparse hash map for block map

2017-03-24 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6471 to look at the new patch set (#4). Change subject: log_block_manager: use sparse_hash_map for block map

[kudu-CR] log block manager: use sparse hash map for block map

2017-03-24 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6471 to look at the new patch set (#3). Change subject: log_block_manager: use sparse_hash_map for block map

[kudu-CR] log block manager: use sparse hash map for block map

2017-03-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log_block_manager: use sparse_hash_map for block map .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6471/1/src/kudu/fs/log_block_manager.h File

[kudu-CR] log block manager: use sparse hash map for block map

2017-03-24 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6471 to look at the new patch set (#2). Change subject: log_block_manager: use sparse_hash_map for block map ..

[kudu-CR] log block manager: use sparse hash map for block map

2017-03-24 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6471 to review the following change. Change subject: log_block_manager: use sparse_hash_map for block map ..