Follow-up from "[MINOR] Fix more missing error checks Vol. 2"
Comments needed at these 4 locations to avoid confusion in the future.
The following discussions from !4836 (merged) should be addressed:
-
@MSevey started a discussion: (+1 comment) Ah yes, why silently ignore errors when you can explicitly ignore them :-)
Should we maybe check these errors?
-
@MSevey started a discussion: (+1 comment) Still not checking the error.
-
@MSevey started a discussion: (+1 comment) Still not checking the error.
-
@MSevey started a discussion: (+2 comments) Should we at least log this error?