Dan Hecht has posted comments on this change.

Change subject: Turn on C++14 in cross-compiled code
......................................................................


Patch Set 2:

> Some of the STL implementation details for maps and iostreams
 > change slightly with the new standard, which has some flow-on
 > effects with how unique ids are assigned and in the functions that
 > use those classes but it doesn't look like there are any other
 > changes. File size is about the same.

Okay, thanks.

-- 
To view, visit http://gerrit.cloudera.org:8080/3024
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If723167b2d03da7592b64a204e31e81ea868e4f2
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to