Skip to content

Commit

Permalink
refactor
Browse files Browse the repository at this point in the history
  • Loading branch information
zhanglei1949 committed Jul 13, 2023
1 parent 293f669 commit 52eb3b7
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions flex/bin/load_plan_and_gen.sh
Original file line number Diff line number Diff line change
Expand Up @@ -119,10 +119,10 @@ cypher_to_plan() {
echo "Fail to find compiler jar."
exit 1
fi

Check warning on line 121 in flex/bin/load_plan_and_gen.sh

View check run for this annotation

codefactor.io / CodeFactor

flex/bin/load_plan_and_gen.sh#L121

Quotes/backslashes in this variable will not be respected. (SC2090)
cmd="java -cp \"${GIE_HOME}/compiler/target/libs/*:${compiler_jar}\""
cmd=${cmd}" -Dgraph.schema=${graph_schema_path}"
cmd=${cmd}" -Djna.library.path=${GIE_HOME}/executor/ir/target/release/"
cmd=${cmd}" com.alibaba.graphscope.common.ir.tools.GraphPlanner \"${ir_compiler_properties}\" \"${real_input_path}\" \"${real_output_path}\""
cmd="java -cp ${GIE_HOME}/compiler/target/libs/*:${compiler_jar}"
cmd="${cmd} -Dgraph.schema=${graph_schema_path}"
cmd="${cmd} -Djna.library.path=${GIE_HOME}/executor/ir/target/release/"
cmd="${cmd} com.alibaba.graphscope.common.ir.tools.GraphPlanner \"${ir_compiler_properties}\" \"${real_input_path}\" \"${real_output_path}\""

Check warning on line 125 in flex/bin/load_plan_and_gen.sh

View check run for this annotation

codefactor.io / CodeFactor

flex/bin/load_plan_and_gen.sh#L125

Quotes/backslashes will be treated literally. Use an array. (SC2089)
echo "running physical plan genration with "${cmd}
eval ${cmd}

Expand Down

0 comments on commit 52eb3b7

Please sign in to comment.