IMPALA-4747: macros should only evaluate their arguments once
The way the macros were previously written, they expanded
so that the 'status' expression was duplicated. This means
if the expression has side-effects, they happen twice.
The fix is to introduce a temporary variable.
Reviewed-by: Matthew Jacobs <email@example.com>
Reviewed-by: Henry Robinson <firstname.lastname@example.org>
Tested-by: Impala Public Jenkins
4 files changed, 20 insertions, 6 deletions
Impala Public Jenkins: Verified
Henry Robinson: Looks good to me, approved
Matthew Jacobs: Looks good to me, but someo