Hello Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6757
to look at the new patch set (#2).
Change subject: IMPALA-5246: UDF's Close() should handle Expr's preparation
failure
Dan Hecht has posted comments on this change.
Change subject: IMPALA-5246: UDF's Close() should handle Expr's preparation
failure
..
Patch Set 1:
Free() already has a null check. What am I missing?
void
Attila Jeges has posted comments on this change.
Change subject: IMPALA-5246: UDF's Close() should handle Expr's preparation
failure
..
Patch Set 1:
(5 comments)
Taras Bobrovytsky has posted comments on this change.
Change subject: IMPALA-5246: UDF's Close() should handle Expr's preparation
failure
..
Patch Set 1: Code-Review+1
(1 comment)
This change makes sense to me.
Michael Ho has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6757
Change subject: IMPALA-5246: UDF's Close() should handle Expr's preparation
failure
..
IMPALA-5246: UDF's Close() should handle Expr's