Hsuan-Yi Chu

Results 6 comments of Hsuan-Yi Chu

Except for the comments, LGTM. Do you run the unit test ?

And can you share the details regarding the performance improvement? For example, how the setting and data look like? Observed % of improvements?

@jcmcote Sorry for the delay. Let me run some tests and commit it if it passes the tests. Your contribution is highly appreciated.

Yes, it passed the tests. I will push it shortly.

Just pushed to master; Thanks for the contributions.

Hi @jcmcote, I see. But can you do us a favor. Regex_replace()'s behavior changes after your patch https://issues.apache.org/jira/browse/DRILL-4645 Can you help fix this? And would you mind merging the missing...