Skip to content

Commit

Permalink
[native] Fix Velox connector configs update
Browse files Browse the repository at this point in the history
The variable needs to be a reference instead of a copy. This is
necessary to ensure that changes made to the config are reflected
in the query context.
  • Loading branch information
zacw7 authored and xiaoxmeng committed Aug 21, 2024
1 parent a868469 commit 6530afc
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ void updateVeloxConnectorConfigs(
const auto& systemConfig = SystemConfig::instance();

for (auto& entry : connectorConfigStrings) {
auto connectorConfig = entry.second;
auto& connectorConfig = entry.second;

// Do not retain cache if `node_selection_strategy` is explicitly set to
// `NO_PREFERENCE`.
Expand Down

0 comments on commit 6530afc

Please sign in to comment.