Istvan Toth

(!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-2119/2/console in case of problems.

(!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-2119/3/console in case of problems.

(!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-2119/4/console in case of problems.

(!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-2119/5/console in case of problems.

(!) A patch to the testing environment has been detected. Re-executing against the patched versions to perform further tests. The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-2119/6/console in case of problems.

There is minimal functionality that we use here. Wouldn't it be better to just get rid of this dependency ?

please fix the checkstyle errors before commit (unless they are pre-existing)

Still +1

Please also fix the markdownlint error.

I have followed up on this, and this won't work with some service definitions. A few service definition, like hbaseui 2.1.0 have a template in the metadata context field. AFAICT...