[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-02 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#6). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] proper handling of catalog shutdown

2017-03-02 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#5). Change subject: [catalog_manager] proper handling of catalog shutdown

[kudu-CR](branch-1.3.x) KUDU-1834 Don't redact KuduRowResult::ToString()

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6244 Change subject: KUDU-1834 Don't redact KuduRowResult::ToString() .. KUDU-1834 Don't redact KuduRowResult::ToString() A user on the client

[kudu-CR] KUDU-1834 Don't redact KuduRowResult::ToString()

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1834 Don't redact KuduRowResult::ToString() .. KUDU-1834 Don't redact KuduRowResult::ToString() A user on the client side calling ToString on the

[kudu-CR](branch-1.3.x) [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6243 Change subject: [security] disable PLAIN SASL mech when authentication is required .. [security] disable PLAIN SASL mech when

[kudu-CR](branch-1.3.x) mapreduce: support for running on secure clusters

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6242 Change subject: mapreduce: support for running on secure clusters .. mapreduce: support for running on secure clusters This adds the

[kudu-CR](branch-1.3.x) ITBLL: avoid creating client many times during Generator step

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6241 Change subject: ITBLL: avoid creating client many times during Generator step .. ITBLL: avoid creating client many times during Generator

[kudu-CR](branch-1.3.x) KUDU-1893 Ensure evaluation of added columns

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6240 Change subject: KUDU-1893 Ensure evaluation of added columns .. KUDU-1893 Ensure evaluation of added columns During a normal scan, a

[kudu-CR] KUDU-1906. Fix lost callback for scanner path

2017-03-02 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6239 to look at the new patch set (#2). Change subject: KUDU-1906. Fix lost callback for scanner path

[kudu-CR] ITBLL: avoid creating client many times during Generator step

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: ITBLL: avoid creating client many times during Generator step .. ITBLL: avoid creating client many times during Generator step Previously ITBLL was

[kudu-CR] ITBLL: avoid creating client many times during Generator step

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: ITBLL: avoid creating client many times during Generator step .. Patch Set 1: Verified+1 Failure was on an unrelated test -- To view, visit

[kudu-CR] mapreduce: support for running on secure clusters

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: mapreduce: support for running on secure clusters .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6237 To unsubscribe,

[kudu-CR] ITBLL: avoid creating client many times during Generator step

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ITBLL: avoid creating client many times during Generator step .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6236 To

[kudu-CR] KUDU-1906. Fix lost callback for scanner path

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1906. Fix lost callback for scanner path .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6239 To unsubscribe, visit

[kudu-CR] KUDU-1906. Fix lost callback for scanner path

2017-03-02 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6239 to review the following change. Change subject: KUDU-1906. Fix lost callback for scanner path .. KUDU-1906.

[kudu-CR] WIP [client] retry RPC if authn token expires

2017-03-02 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6238 to look at the new patch set (#2). Change subject: WIP [client] retry RPC if authn token expires .. WIP [client]

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [util] conventional signature for Status::operator=() .. [util] conventional signature for Status::operator=() Updated the signature of the

[kudu-CR] [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [security] disable PLAIN SASL mech when authentication is required .. [security] disable PLAIN SASL mech when authentication is required Change-Id:

[kudu-CR] WIP [client] retry RPC if authn token expires

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/6238 Change subject: WIP [client] retry RPC if authn token expires .. WIP [client] retry RPC if authn token expires Get a new authn token

[kudu-CR] [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [security] disable PLAIN SASL mech when authentication is required .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6235 To

[kudu-CR] [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6235 to look at the new patch set (#3). Change subject: [security] disable PLAIN SASL mech when authentication is required

[kudu-CR] [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [security] disable PLAIN SASL mech when authentication is required .. Patch Set 3: (1 comment)

[kudu-CR](branch-1.3.x) server: offer a flag to completely disable the web server

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: server: offer a flag to completely disable the web server .. server: offer a flag to completely disable the web server This adds a new

[kudu-CR](branch-1.3.x) server: offer a flag to completely disable the web server

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: server: offer a flag to completely disable the web server .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6234 To unsubscribe,

[kudu-CR] [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [security] disable PLAIN SASL mech when authentication is required .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] ITBLL: avoid creating client many times during Generator step

2017-03-02 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6236 to review the following change. Change subject: ITBLL: avoid creating client many times during Generator step

[kudu-CR] mapreduce: support for running on secure clusters

2017-03-02 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6237 to review the following change. Change subject: mapreduce: support for running on secure clusters

[kudu-CR] [security] disable PLAIN SASL mech when authentication is required

2017-03-02 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: [security] disable PLAIN SASL mech when authentication is required .. [security] disable PLAIN SASL mech when authentication is required Change-Id:

[kudu-CR] [security] PLAIN SASL mech when authentication is required

2017-03-02 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6235 to review the following change. Change subject: [security] PLAIN SASL mech when authentication is required

[kudu-CR](branch-1.3.x) webserver: add X-Frame-Options header

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: webserver: add X-Frame-Options header .. webserver: add X-Frame-Options header This adds a default 'DENY' header in order to prevent Kudu web pages from

[kudu-CR](branch-1.3.x) webserver: add X-Frame-Options header

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: webserver: add X-Frame-Options header .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6233 To unsubscribe, visit

[kudu-CR] [flaky tests] Allow for the master to take longer to start

2017-03-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: [flaky tests] Allow for the master to take longer to start .. [flaky tests] Allow for the master to take longer to start With the additional work

[kudu-CR] webserver: add X-Frame-Options header

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: webserver: add X-Frame-Options header .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6215/3/src/kudu/server/webserver.cc File

[kudu-CR] webserver: add X-Frame-Options header

2017-03-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: webserver: add X-Frame-Options header .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6215/3/src/kudu/server/webserver.cc File

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [util] conventional signature for Status::operator=() .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6175 To unsubscribe, visit

[kudu-CR] [ts recovery-itest] Reduce the number of rows written in TestChangeMaxCellSize

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [ts_recovery-itest] Reduce the number of rows written in TestChangeMaxCellSize .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [ts recovery-itest] Reduce the number of rows written in TestChangeMaxCellSize

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [ts_recovery-itest] Reduce the number of rows written in TestChangeMaxCellSize .. [ts_recovery-itest] Reduce the number of rows written in

[kudu-CR](branch-1.3.x) server: offer a flag to completely disable the web server

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6234 Change subject: server: offer a flag to completely disable the web server .. server: offer a flag to completely disable the web server

[kudu-CR] server: offer a flag to completely disable the web server

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: server: offer a flag to completely disable the web server .. server: offer a flag to completely disable the web server This adds a new

[kudu-CR] server: offer a flag to completely disable the web server

2017-03-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: server: offer a flag to completely disable the web server .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6216 To unsubscribe,

[kudu-CR](branch-1.3.x) webserver: add X-Frame-Options header

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6233 Change subject: webserver: add X-Frame-Options header .. webserver: add X-Frame-Options header This adds a default 'DENY' header in order

[kudu-CR] webserver: add X-Frame-Options header

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: webserver: add X-Frame-Options header .. webserver: add X-Frame-Options header This adds a default 'DENY' header in order to prevent Kudu web pages from

[kudu-CR] webserver: add X-Frame-Options header

2017-03-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: webserver: add X-Frame-Options header .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6215 To unsubscribe, visit

[kudu-CR] Make ExternalDaemon::StartProcess() handle fault injection

2017-03-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6211 to look at the new patch set (#6). Change subject: Make ExternalDaemon::StartProcess() handle fault injection ..

[kudu-CR] [flaky tests] Allow for the master to take longer to start

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [flaky tests] Allow for the master to take longer to start .. Patch Set 1: Code-Review+2 lgtm, curious to know what's taking so long at boot, maybe we should add

[kudu-CR] [flaky tests] Allow for the master to take longer to start

2017-03-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6230 Change subject: [flaky tests] Allow for the master to take longer to start .. [flaky tests] Allow for the master to take longer to

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] conventional signature for Status::operator=() .. Patch Set 3: > > The symbol name's the same, but does a program built against the > > old SO still run

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] conventional signature for Status::operator=() .. Patch Set 3: > The symbol name's the same, but does a program built against the > old SO still run

[kudu-CR](branch-1.3.x) [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6229 To

[kudu-CR](branch-1.3.x) [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. [security] Add per-connection nonce for Kerberos replay resistance Kerberos is

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [util] conventional signature for Status::operator=() .. Patch Set 3: The symbol name's the same, but does a program built against the old SO still run properly

[kudu-CR] Make ExternalDaemon::StartProcess() handle fault injection

2017-03-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6211 to look at the new patch set (#4). Change subject: Make ExternalDaemon::StartProcess() handle fault injection ..

[kudu-CR] Make ExternalDaemon::StartProcess() handle fault injection

2017-03-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6211 to look at the new patch set (#5). Change subject: Make ExternalDaemon::StartProcess() handle fault injection ..

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [util] conventional signature for Status::operator=() .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6175 To unsubscribe, visit

[kudu-CR] Update version to 1.4.0-SNAPSHOT

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Update version to 1.4.0-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6223 To unsubscribe, visit

[kudu-CR] Update version to 1.4.0-SNAPSHOT

2017-03-02 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Update version to 1.4.0-SNAPSHOT .. Update version to 1.4.0-SNAPSHOT Change-Id: I128157c46f6c8f77ecee604591619917f5d1ce42 Reviewed-on:

[kudu-CR](branch-1.3.x) [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/6229 Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. [security] Add per-connection nonce for Kerberos

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. [security] Add per-connection nonce for Kerberos replay resistance Kerberos is

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6137 To

[kudu-CR] [ts recovery-itest] Reduce the number of rows written in TestChangeMaxCellSize

2017-03-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: [ts_recovery-itest] Reduce the number of rows written in TestChangeMaxCellSize .. [ts_recovery-itest] Reduce the number of rows written in

[kudu-CR] [ts recovery-itest] Reduce the number of rows written

2017-03-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6227 Change subject: [ts_recovery-itest] Reduce the number of rows written .. [ts_recovery-itest] Reduce the number of rows written

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. Patch Set 10: Code-Review+1 (1 comment)

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6137 to look at the new patch set (#10). Change subject: [security] Add per-connection nonce for Kerberos replay resistance

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [security] Add per-connection nonce for Kerberos replay resistance .. Patch Set 7: (10 comments) http://gerrit.cloudera.org:8080/#/c/6137/7//COMMIT_MSG Commit

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6137 to look at the new patch set (#9). Change subject: [security] Add per-connection nonce for Kerberos replay resistance

[kudu-CR] [security] Add per-connection nonce for Kerberos replay resistance

2017-03-02 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6137 to look at the new patch set (#8). Change subject: [security] Add per-connection nonce for Kerberos replay resistance

[kudu-CR] [util] conventional signature for Status::operator=()

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] conventional signature for Status::operator=() .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6175/2/src/kudu/util/status.h File

[kudu-CR](branch-1.2.x) KUDU-1893 Ensure evaluation of added columns

2017-03-02 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-1893 Ensure evaluation of added columns .. Patch Set 1: (7 comments) Posted differences between this and master:kudu-1893.

[kudu-CR] Make ExternalDaemon::StartProcess() handle fault injection

2017-03-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6211 to look at the new patch set (#3). Change subject: Make ExternalDaemon::StartProcess() handle fault injection ..

[kudu-CR] Make ExternalDaemon::StartProcess() handle fault injection

2017-03-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6211 to look at the new patch set (#2). Change subject: Make ExternalDaemon::StartProcess() handle fault injection ..

[kudu-CR](branch-1.2.x) KUDU-1893 Ensure evaluation of added columns

2017-03-02 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/6225 Change subject: KUDU-1893 Ensure evaluation of added columns .. KUDU-1893 Ensure evaluation of added columns During a normal scan, a

[kudu-CR] [system catalog] an option to reset CA entries in system table

2017-03-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: [system_catalog] an option to reset CA entries in system table .. Abandoned If needed, should be done as a separate utility. -- To view, visit

[kudu-CR] KUDU-1890 Allow renaming of primary key column

2017-03-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1890 Allow renaming of primary key column .. Patch Set 7: (6 comments) http://gerrit.cloudera.org:8080/#/c/6078/7/docs/known_issues.adoc File

[kudu-CR] KUDU-1893 Ensure evaluation of added columns

2017-03-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-1893 Ensure evaluation of added columns .. KUDU-1893 Ensure evaluation of added columns During a normal scan, a CFileIterator sets a flag to