You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2022/08/25 03:47:52 UTC

[GitHub] [iotdb] HTHou commented on a diff in pull request #7119: [To rel/0.13][IOTDB-4230]add checking iotdb.in.sh file, add iotdb_log_dir system variable

HTHou commented on code in PR #7119:
URL: https://github.com/apache/iotdb/pull/7119#discussion_r954474275


##########
server/src/assembly/resources/sbin/start-server.sh:
##########
@@ -23,12 +23,37 @@ echo ---------------------
 echo Starting IoTDB
 echo ---------------------
 
+
+if [ "x$IOTDB_INCLUDE" = "x" ]; then
+    # Locations (in order) to use when searching for an include file.
+    for include in "`dirname "$0"`/iotdb.in.sh" \
+                   "$HOME/.iotdb.in.sh" \
+                   /usr/share/iotdb/iotdb.in.sh \
+                   /etc/iotdb/iotdb.in.sh \
+                   /opt/iotdb/iotdb.in.sh; do
+        if [ -r "$include" ]; then
+            . "$include"
+            break
+        fi
+    done
+# ...otherwise, source the specified include.
+elif [ -r "$CASSANDRA_INCLUDE" ]; then
+    . "$CASSANDRA_INCLUDE"

Review Comment:
   ```suggestion
   elif [ -r "$IOTDB_INCLUDE" ]; then
       . "$IOTDB_INCLUDE"
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org