Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22573 )

Change subject: KUDU-3646 fix Base64Decode()
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22573/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22573/3//COMMIT_MSG@7
PS3, Line 7: fix Base64Decode()
> Ah, that's a good point, indeed.  I've just filed KUDU-3648 to reflect on a
If you are interested, below is just another scenario that helps exposing the 
issue:
  1. Apply diff from https://gerrit.cloudera.org/#/c/22568/ on top of the main 
branch (NOTE: apply the diff only, not checking out the base version that 
includes this patch as well).
  2. Build Kudu C++ components and tests.
  3. From the build directory, run ./bin/hms_notification_log_listener-test



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7f5d41eb4e41897d520f00b12c6e487ecc5a00fc
Gerrit-Change-Number: 22573
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Comment-Date: Fri, 07 Mar 2025 18:46:13 +0000
Gerrit-HasComments: Yes

Reply via email to