[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 10: Code-Review+2 Carrying over Adar's +2 from PS8. -- To view, visit http:/

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. [util] introduce TTL cache This changelist introduces TTL cache based on the cac

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/12825/7/src/kudu/util/ttl

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12825 to look at the new patch set (#10). Change subject: [util] introduce TTL cache .

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/12825/7//COMMIT_MSG Commi

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 9: Code-Review+1 (3 comments) LGTM, just some minor nits. http://gerrit.cloudera.o

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12825 to look at the new patch set (#9). Change subject: [util] introduce TTL cache ..

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 9: (4 comments) > Patch Set 8: Code-Review+2 > > (4 comments) http://gerrit.

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 8: Code-Review+2 (4 comments) http://gerrit.cloudera.org:8080/#/c/12825/8/src/ku

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 7: (4 comments) http://gerrit.cloudera.org:8080/#/c/12825/7/src/kudu/util/ttl

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12825 to look at the new patch set (#8). Change subject: [util] introduce TTL cache ..

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 7: Code-Review+2 (4 comments) http://gerrit.cloudera.org:8080/#/c/12825/7/src/ku

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 7: (7 comments) http://gerrit.cloudera.org:8080/#/c/12825/6/src/kudu/util/ttl

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12825 to look at the new patch set (#7). Change subject: [util] introduce TTL cache ..

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/12825/4/src/kudu/util/ttl_cache.

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 6: (6 comments) http://gerrit.cloudera.org:8080/#/c/12825/6/src/kudu/util/ttl_ca

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/12825/4/src/kudu/util/ttl_

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/12825/4/src/kudu/util/ttl_

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/12825/4/src/kudu/util/ttl_cache.

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12825 to look at the new patch set (#6). Change subject: [util] introduce TTL cache ..

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12825 ) Change subject: [util] introduce TTL cache .. Patch Set 4: (7 comments) > Also is this still WIP? Do you intend on adding the TTL-met

[kudu-CR] [util] introduce TTL cache

2019-03-28 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12825 to look at the new patch set (#5). Change subject: [util] introduce TTL cache ..