Skip to content

Commit

Permalink
[Enhancement] Turn on the default starcache UT switch to build the st…
Browse files Browse the repository at this point in the history
…arrocks unittests with datacache. (#34638)

Signed-off-by: Gavin <[email protected]>
  • Loading branch information
GavinMar authored Nov 9, 2023
1 parent 8dfc8b5 commit 1e6cb9d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion gensrc/thrift/InternalService.thrift
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ struct TPlanFragmentExecParams {
// To enable pass through chunks between sink/exchange if they are in the same process.
52: optional bool enable_exchange_pass_through

53: optional map<Types.TPlanNodeId, map<i32, list<TScanRangeParams>>> node_to_per_driver_seq_scan_ranges
53: optional map<Types.TPlanNodeId, map<i32, list<TScanRangeParams>>> node_to_per_driver_seq_scan_ranges

54: optional bool enable_exchange_perf

Expand Down
4 changes: 1 addition & 3 deletions run-be-ut.sh
Original file line number Diff line number Diff line change
Expand Up @@ -152,10 +152,8 @@ if [ "${USE_STAROS}" == "ON" ]; then
export STARLET_INSTALL_DIR
fi

# Temporarily keep the default behavior same as before to avoid frequent thirdparty update.
# Once the starcache version is stable, we will turn on it by default.
if [[ -z ${WITH_STARCACHE} ]]; then
WITH_STARCACHE=${USE_STAROS}
WITH_STARCACHE=ON
fi

${CMAKE_CMD} -G "${CMAKE_GENERATOR}" \
Expand Down

0 comments on commit 1e6cb9d

Please sign in to comment.