[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] [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] [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] [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] [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] [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] [util] conventional signature for Status::operator=()

2017-02-28 Thread Adar Dembo (Code Review)
Adar Dembo 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] [util] conventional signature for Status::operator=()

2017-02-28 Thread Todd Lipcon (Code Review)
Todd Lipcon 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] [util] conventional signature for Status::operator=()

2017-02-27 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6175 to look at the new patch set (#3). Change subject: [util] conventional signature for Status::operator=() .. [util]

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

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

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

2017-02-27 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6175 to look at the new patch set (#2). Change subject: [util] conventional signature for Status::operator=() .. [util]

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

2017-02-27 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/6175 Change subject: [util] conventional signature for Status::operator=() .. [util] conventional signature for Status::operator=() Updated