[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-30 Thread Henry Robinson (Code Review)
Henry Robinson has submitted this change and it was merged. Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 Thrift commit taken from:

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-30 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. Patch Set 5: Code-Review+2 Verified+1 Verified by a toolchain build. -- To view, visit http://

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-29 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. Patch Set 4: (1 comment) Also fix a bug in how the options are checked after setting. http://g

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-29 Thread Henry Robinson (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7859 to look at the new patch set (#5). Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 ..

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-29 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. Patch Set 4: Code-Review+2 (1 comment) LGTM, just need to add a comment as mentioned below, so i

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-29 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7859/4/source/thrift/thrift-0.9.0-

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-29 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7859/4/source/thrift/thrift-0.9.0-p

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-28 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#4). Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 Thrift commit taken from: https://gi

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-28 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#3). Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 Thrift commit taken from: https://gi

[native-toolchain-CR] IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0

2017-08-28 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7859 Change subject: IMPALA-5849: Disable compile-time checks for OpenSSL > 1.0.0 .. IMPALA-5849: Disable compile-time checks for OpenSSL > 1