Skip to content

Commit d35a70d

Browse files
authored
Merge pull request #1312 from sqlancer/fix/datafusion-server-memory-leak
Fix DataFusion server memory leak: clear state on database reset
2 parents 5789b2a + c548689 commit d35a70d

File tree

1 file changed

+5
-0
lines changed
  • src/sqlancer/datafusion/server/datafusion_server/src

1 file changed

+5
-0
lines changed

src/sqlancer/datafusion/server/datafusion_server/src/main.rs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,11 @@ impl FlightSqlService for FlightSqlServiceImpl {
215215

216216
let mut ctx_guard = self.ctx.lock().await; // Use `lock()` for async Mutex
217217
*ctx_guard = new_ctx;
218+
219+
// Clear leaked state from previous round
220+
self.statements.clear();
221+
self.results.clear();
222+
self.contexts.clear();
218223
}
219224
// no authentication actually takes place here
220225
// see Ballista implementation for example of basic auth

0 commit comments

Comments
 (0)