Skip to content

Commit 09510df

Browse files
committed
fix issue with variable assignment
1 parent ca7d13b commit 09510df

File tree

2 files changed

+5
-10
lines changed

2 files changed

+5
-10
lines changed

ballista/core/src/client.rs

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,7 @@ const IO_RETRY_WAIT_TIME_MS: u64 = 3000;
6161
impl BallistaClient {
6262
/// Create a new BallistaClient to connect to the executor listening on the specified
6363
/// host and port
64-
pub async fn try_new(
65-
host: &str,
66-
port: u16,
67-
grpc_client_max_message_size: usize,
68-
) -> Result<Self> {
64+
pub async fn try_new(host: &str, port: u16, max_message_size: usize) -> Result<Self> {
6965
let addr = format!("http://{host}:{port}");
7066
debug!("BallistaClient connecting to {}", addr);
7167
let connection =
@@ -77,8 +73,8 @@ impl BallistaClient {
7773
))
7874
})?;
7975
let flight_client = FlightServiceClient::new(connection)
80-
.max_decoding_message_size(grpc_client_max_message_size)
81-
.max_encoding_message_size(grpc_client_max_message_size);
76+
.max_decoding_message_size(max_message_size)
77+
.max_encoding_message_size(max_message_size);
8278

8379
debug!("BallistaClient connected OK: {:?}", flight_client);
8480

ballista/core/src/execution_plans/shuffle_reader.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -149,9 +149,8 @@ impl ExecutionPlan for ShuffleReaderExec {
149149

150150
let config = context.session_config();
151151

152-
let max_request_num = config.ballista_grpc_client_max_message_size();
153-
let max_message_size =
154-
config.ballista_shuffle_reader_maximum_in_flight_requests();
152+
let max_request_num = config.ballista_shuffle_reader_maximum_in_flight_requests();
153+
let max_message_size = config.ballista_grpc_client_max_message_size();
155154

156155
log::debug!(
157156
"ShuffleReaderExec::execute({}) max_request_num: {}, max_message_size: {}",

0 commit comments

Comments
 (0)