[GitHub] [spark] MaxGekk commented on pull request #40970: [SPARK-43290][SQL] Adds IV and AAD support to aes_encrypt/aes_decrypt

2023-05-21 Thread via GitHub
MaxGekk commented on PR #40970: URL: https://github.com/apache/spark/pull/40970#issuecomment-1556111345 > I want to land this first. ok. Let's modify PR's title and its description according to your actual changes. -- This is an automated message from the Apache Git Service. To

[GitHub] [spark] MaxGekk commented on pull request #40970: [SPARK-43290][SQL] Adds IV and AAD support to aes_encrypt/aes_decrypt

2023-05-17 Thread via GitHub
MaxGekk commented on PR #40970: URL: https://github.com/apache/spark/pull/40970#issuecomment-1550789731 > This change adds support for optional IV and AAD fields to aes_encrypt and aes_decrypt @sweisdb Looking at the constructors of the `AesEncrypt` and `AesDecrypt` expressions,

[GitHub] [spark] MaxGekk commented on pull request #40970: [SPARK-43290][SQL] Adds IV and AAD support to aes_encrypt/aes_decrypt

2023-05-16 Thread via GitHub
MaxGekk commented on PR #40970: URL: https://github.com/apache/spark/pull/40970#issuecomment-1549417703 @sweisdb Could you fix the code style issues: ``` Checkstyle checks failed at following occurrences: Error: