[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6624 to look at the new patch set (#4). Change subject: WIP: Expose a way to set "advanced" non-types scan options ..

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [mini-kdc] adapted config for krb5 1.15 .. Patch Set 1: > Do we need this in the Java kdc also? Ugh, sure. I haven't run Java tests since upgrading my

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-13 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 1: This is just a quick push to sanity check my approach. -- To view, visit

[kudu-CR] Allow to release an rpc transfer's data

2017-04-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow to release an rpc transfer's data .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/6592/8/src/kudu/rpc/outbound_call.h File

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-13 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6630 Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add checksumming to cfile Adds optional checksumming and

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

2017-04-13 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#2). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [mini-kdc] adapted config for krb5 1.15 .. Patch Set 1: Do we need this in the Java kdc also? -- To view, visit http://gerrit.cloudera.org:8080/6629 To

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6629 to look at the new patch set (#2). Change subject: [mini-kdc] adapted config for krb5 1.15 .. [mini-kdc] adapted

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit Message:

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [mini-kdc] adapted config for krb5 1.15 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6629 To unsubscribe, visit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit Message:

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (4 comments)

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit Message:

[kudu-CR] [security] avoid sparse seq numbers in TokenSigner

2017-04-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [security] avoid sparse seq numbers in TokenSigner .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6329/1/src/kudu/security/token-test.cc File

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: use extent maps to decide whether to truncate containers .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] fs: generate report during Open

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/log_block_manager-test.cc File

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment)

[kudu-CR] [security] avoid sparse seq numbers in TokenSigner

2017-04-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] avoid sparse seq numbers in TokenSigner .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6329/1/src/kudu/security/token-test.cc File

[kudu-CR] [security] avoid sparse seq numbers in TokenSigner

2017-04-13 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6329 to look at the new patch set (#2). Change subject: [security] avoid sparse seq numbers in TokenSigner ..

[kudu-CR] log block manager: corruptor test utility

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] env: add RWFile::GetExtentMap for analyzing file extents

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: env: add RWFile::GetExtentMap for analyzing file extents .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6583 To

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [mini-kdc] adapted config for krb5 1.15 .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6629 To unsubscribe, visit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit Message:

[kudu-CR] fs: generate report during Open

2017-04-13 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#7). Change subject: fs: generate report during Open .. fs: generate report during

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-13 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6585 to look at the new patch set (#6). Change subject: log block manager: use extent maps to decide whether to truncate containers

[kudu-CR] fs: generate report during Open

2017-04-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/log_block_manager-test.cc File

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit Message:

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log block manager: use extent maps to decide whether to truncate containers .. Patch Set 6: Have you done any tests of how this affects startup time,

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment)

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment)

[kudu-CR] [security] avoid sparse seq numbers in TokenSigner

2017-04-13 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6329 to look at the new patch set (#3). Change subject: [security] avoid sparse seq numbers in TokenSigner ..

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [mini-kdc] adapted config for krb5 1.15 .. Patch Set 2: Unrelated error due to change in error message -- will address that separately. -- To view, visit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit Message:

[kudu-CR] log block manager: corruptor test utility

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 7: Code-Review+2 (2 comments)

[kudu-CR] [mini-kdc] adapted config for krb5 1.15

2017-04-13 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/6629 Change subject: [mini-kdc] adapted config for krb5 1.15 .. [mini-kdc] adapted config for krb5 1.15 Starting with version 1.15 of