Commits


Gidon Gershinsky authored and Wes McKinney committed f199a77c10e
PARQUET-1517: [C++] Crypto package updates to match the final spec An initial version of crypto package is merged. This PR updates the crypto code to conform the signed off specification (wire protocol updates, signature tag creation, AAD support, etc) improve performance by extending cipher lifecycle to file writing/reading - instead of creating cipher on each encrypt/decrypt operation Author: Gidon Gershinsky <gg5070@gmail.com> Author: Revital1 Eres <eres@iris-tes-cloud.sl.cloud9.ibm.com> Author: Gidon Gershinsky <gidon@il.ibm.com> Author: Revital Sur <eres@il.ibm.com> Author: thamht4190 <thamht01188@gmail.com> Author: ggershinsky <ggershinsky@users.noreply.github.com> Closes #3520 from ggershinsky/p1517-crypto-pack-updates and squashes the following commits: 21ce9d097 <ggershinsky> Merge pull request #6 from revital76/review_comments ef970e3d8 <Revital1 Eres> Fix broken line 3ffd60645 <Revital1 Eres> Change comment in encryption_internal.h b570e8e9b <Revital1 Eres> Fixes following Gidon's comments 535d0e28c <Revital1 Eres> Delete encryption_internal.cc from CMakeLists.txt 9be898eb3 <Revital1 Eres> Address review comments e784d9d9c <Gidon Gershinsky> cipher wipe out abd76a622 <thamht4190> fix build issue on MacOS 24795fae1 <Gidon Gershinsky> ctr fixes 6c599e936 <thamht4190> fix code style 9fa9ef665 <Gidon Gershinsky> rm old method 9f68cab27 <Gidon Gershinsky> encryption size delta 4d832d66b <Gidon Gershinsky> stateful encryptor objects 73c8235ea <Revital Sur> Fix indentation in crypto.h and crypto.cc 333045b79 <Revital Sur> Add functions for AAD calculation and adjust code for API changes aa7b2abd5 <Gidon Gershinsky> params order 1a9972538 <Gidon Gershinsky> cast fix dfe98ee7c <Gidon Gershinsky> signed footer encryption 8961544bb <Gidon Gershinsky> CTR IV fix 7ecdade4f <Gidon Gershinsky> iv comment and buffer length 8e7fe9021 <Gidon Gershinsky> set or check ciphertext length 03ede654f <Gidon Gershinsky> iv changes and buffer length