[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15981 to look at the new patch set (#4). Change subject: replace shared_ptr(new T) with make_shared()

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15981 to look at the new patch set (#3). Change subject: replace shared_ptr(new T) with make_shared()

[kudu-CR] KUDU-1865 (part 1): reduce some cross-thread allocations

2020-05-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has restored this change. ( http://gerrit.cloudera.org:8080/5905 ) Change subject: KUDU-1865 (part 1): reduce some cross-thread allocations .. Restored -- To view, visit http://gerrit.cloudera.org:8080/5905 To

[kudu-CR] WIP: KUDU-1865 (part 1): reduce some cross-thread allocations

2020-05-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#5) to the change originally created by Todd Lipcon. ( http://gerrit.cloudera.org:8080/5905 ) Change subject: WIP: KUDU-1865 (part 1): reduce some cross-thread allocations .. WIP:

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15981 Change subject: replace shared_ptr(new T) with make_shared() .. replace shared_ptr(new T) with make_shared() This patch replaces

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15981 ) Change subject: replace shared_ptr(new T) with make_shared() .. Patch Set 2: (1 comment)

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15981 ) Change subject: replace shared_ptr(new T) with make_shared() .. Patch Set 2: (1 comment)

[kudu-CR] [client] Remove allocator param for direct Bloom filter predicate

2020-05-22 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15947 ) Change subject: [client] Remove allocator param for direct Bloom filter predicate .. Patch Set 3: (1 comment)

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15981 ) Change subject: replace shared_ptr(new T) with make_shared() .. Patch Set 2: (1 comment)

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Bankim Bhavsar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15981 to look at the new patch set (#2). Change subject: replace shared_ptr(new T) with make_shared()

[kudu-CR] replace shared ptr(new T) with make shared()

2020-05-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15981 ) Change subject: replace shared_ptr(new T) with make_shared() .. Patch Set 2: (1 comment)