This is an automated email from the ASF dual-hosted git repository.

xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 0500c12  Fixing pinot-admin script to parse env var PLUGINS_INCLUDE 
(#7145)
0500c12 is described below

commit 0500c121a115b4412b7a5b08452359e76d93b095
Author: Xiang Fu <xiangfu.1...@gmail.com>
AuthorDate: Fri Jul 9 14:14:50 2021 -0700

    Fixing pinot-admin script to parse env var PLUGINS_INCLUDE (#7145)
---
 pinot-tools/src/main/resources/appAssemblerScriptTemplate | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/pinot-tools/src/main/resources/appAssemblerScriptTemplate 
b/pinot-tools/src/main/resources/appAssemblerScriptTemplate
index 1f8b2e9..e3c7459 100644
--- a/pinot-tools/src/main/resources/appAssemblerScriptTemplate
+++ b/pinot-tools/src/main/resources/appAssemblerScriptTemplate
@@ -117,8 +117,8 @@ if [ -z "$PLUGINS_CLASSPATH" ] ; then
   fi
   if [ -d "$PLUGINS_DIR" ] ; then
     if [ -n "$PLUGINS_INCLUDE" ] ; then
-      IFS=';' echo "$PLUGINS_INCLUDE" | read -ra PLUGINS_ARR
-      for PLUGIN_JAR in "${PLUGINS_ARR[@]}"; do
+      export IFS=";"
+      for PLUGIN_JAR in $PLUGINS_INCLUDE; do
           PLUGIN_JAR_PATH=$(find "$PLUGINS_DIR" -path 
\*/"$PLUGIN_JAR"/"$PLUGIN_JAR"-\*.jar)
           if [ -n "$PLUGINS_CLASSPATH" ] ; then
             PLUGINS_CLASSPATH=$PLUGINS_CLASSPATH:$PLUGIN_JAR_PATH
@@ -126,6 +126,7 @@ if [ -z "$PLUGINS_CLASSPATH" ] ; then
             PLUGINS_CLASSPATH=$PLUGIN_JAR_PATH
           fi
       done
+      unset IFS
     else
       PLUGIN_JARS=$(find "$PLUGINS_DIR" -name \*.jar)
       for PLUGIN_JAR in $PLUGIN_JARS ; do

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to