Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12845 )
Change subject: Re-land IMPALA-5393. Use THREAD_LOCAL state for regexp ...................................................................... Patch Set 1: > Patch Set 1: > > Yeah FWIW I think Lars did the right thing in erring on the side of reverting > in this case. yep, when in doubt, revert to keep builds green for sure -- To view, visit http://gerrit.cloudera.org:8080/12845 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9ae0703efeb2429813b2a712f1accf1b0a4a409e Gerrit-Change-Number: 12845 Gerrit-PatchSet: 1 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Mon, 25 Mar 2019 20:49:50 +0000 Gerrit-HasComments: No