fixing ci

alibaba · May 24, 2024 · 4f3c814 · 4f3c814
1 parent 5ba1b31
commit 4f3c814
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 7 deletions.
diff --git a/.github/workflows/hqps-db-ci.yml b/.github/workflows/hqps-db-ci.yml
@@ -147,8 +147,10 @@ jobs:
         cd java
         mvn clean install -DskipTests
         cd ../python
+        pip3 install -r requirements.txt
         pip3 install -r test-requirements.txt
-        pip3 install .
+        export PATH=${HOME}/.local/bin:$PATH
+        python3 setup.py build_proto
 
         rm -rf ${TMP_INTERACTIVE_WORKSPACE}     
         cd ${GITHUB_WORKSPACE}/flex/build/

diff --git a/.gitignore b/.gitignore
@@ -134,6 +134,7 @@ flex/interactive/sdk/python/.travis.yml
 flex/interactive/sdk/python/tox.ini
 flex/interactive/sdk/python/pyproject.toml
 flex/interactive/sdk/python/interactive_sdk/openapi/
+flex/interactive/sdk/python/interactive_sdk/client/generated/__init__.py
 
 
 **/.cache/
diff --git a/flex/engines/hqps_db/app/interactive_app_base.h b/flex/engines/hqps_db/app/interactive_app_base.h
@@ -65,7 +65,7 @@ bool deserialize(std::tuple<ARGS...>& tuple, std::string_view sv) {
   auto j = nlohmann::json::parse(sv);
   if (!j.contains("arguments")) {
     LOG(ERROR) << "No arguments found in input";
-    return false;
+    return sizeof...(ARGS) == 0;
   }
   auto arguments_list = j["arguments"];
   if (arguments_list.is_array()) {

diff --git a/flex/tests/hqps/engine_config_test.yaml b/flex/tests/hqps/engine_config_test.yaml
@@ -5,7 +5,7 @@ directories:
     logs: logs
     conf: conf
 log_level: INFO
-default_graph: modern_graph
+default_graph: ldbc
 compute_engine:
   type: hiactor
   workers:

diff --git a/flex/tests/hqps/hqps_sdk_test.sh b/flex/tests/hqps/hqps_sdk_test.sh
@@ -103,10 +103,6 @@ run_java_sdk_test(){
 run_python_sdk_test(){
   echo "run python sdk test"
   pushd ${FLEX_HOME}/interactive/sdk/python/
-  pip3 install -r requirements.txt 
-  pip3 install -r test-requirements.txt
-  export PATH=${HOME}/.local/bin:$PATH
-  python3 setup.py build_proto
   cmd="python3 -m pytest -s test/test_driver.py"
   echo "Start python sdk test: ${cmd}"
   eval ${cmd} || (err "java python test failed" &&  exit 1)