I've merged this to main. (Thanks for the review, Zhe Hu!) There are new lint checks on code (especially javadoc) and commit messages. This may cause existing open PRs to give errors. Sorry about that, but it should help us all write better commit messages. (And reduce nagging from reviewers like me.)
Julian On Wed, Jun 7, 2023 at 2:34 PM Julian Hyde <jh...@apache.org> wrote: > > Can I please get a review for PR 3250? It has several commits that are > loosely connected. > > CALCITE-5762 should be reviewed from a security perspective (because > it deals with unsafe methods such as ProcessBuilder). > > CALCITE-5764 adds a fun class called Puffin that brings the scripting > capabilities of 'awk' into Java. > > https://github.com/apache/calcite/pull/3250 > > Julian