Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/17251 )
Change subject: [encryption] Add encryption support to Env ...................................................................... Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/17251/9/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: http://gerrit.cloudera.org:8080/#/c/17251/9/src/kudu/util/env_posix.cc@597 PS9, Line 597: RETURN_NOT_OK(DoEncryptV(kDummyEncryptionKey, offset, data, encrypted_data); : encrypted_buf.release(); > That's right, but from what I can see these two snippets are equivalent in Sorry, I think I misunderstood, this is a good idea, I didn't think of it, thanks for the suggestion. -- To view, visit http://gerrit.cloudera.org:8080/17251 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id5a81b5736a0c1970818dbbc4a234480476b8d41 Gerrit-Change-Number: 17251 Gerrit-PatchSet: 11 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sat, 11 Sep 2021 07:05:28 +0000 Gerrit-HasComments: Yes
