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

Change subject: [thirdparty] handle OpenSSL errors properly in curl and squeasel
......................................................................


Patch Set 1:

> Thanks for this patch! A small nit, there are a few leftover
 > whitespaces in the diffs, I'm not sure if those were already
 > present in the original files, or just part of the diff format.

As for the spaces that gerrit highlights in the beginning of lines: that's how 
diff files look like, there is no need to fix that :)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I209ceb35ded66222c90b82e98e7d191f33573217
Gerrit-Change-Number: 19849
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Fri, 05 May 2023 16:52:11 +0000
Gerrit-HasComments: No

Reply via email to