feat: Implement Arrow Flight Middleware support#139
feat: Implement Arrow Flight Middleware support#139CurtHagenlocher merged 3 commits intoapache:mainfrom
Conversation
|
@CurtHagenlocher |
I'm not great at Git/Github strategies. If I were you, I'd probably get a copy of my branch by cloning it and then just overwriting the files in a branch of your own. |
|
@CurtHagenlocher I ran dotnet build -p:EnforceCodeStyleInBuild=true and didn’t see any issues. Which tool/command should I run to surface the linting errors? If there aren’t any, could you please merge it? |
Oh, I'm sorry if that wasn't clear but I'd fixed the linter errors on my branch in order to get the PR to green. |
|
@CurtHagenlocher Hi Curt, can you please merge the PR then? I'm waiting for a year for this to be merged. |
Enhancement: Apache Arrow Flight Middleware in C#
Co-authored-by: @HackPoint
Original change in apache/arrow#46316.
Overview
This Pull Request enhances middleware support for Apache Arrow Flight using C#, focusing on improved metadata header management and propagation for better observability and extensibility. It also provides handling for HTTP/HTTPS communication.
Rationale for this Change
Effective middleware is critical for managing metadata headers, ensuring accurate request/response handling, and simplifying debugging in distributed systems. By improving middleware capabilities, we enhance reliability and observability, significantly benefiting developers and operational teams managing complex Flight-based applications.
What's Included in this PR?
Key Features
OnBeforeSendingHeaders,OnHeadersReceived,OnCallCompleted).Impact
Are These Changes Tested?
Testing Overview
Unit Tests:
OnBeforeSendingHeaders,OnHeadersReceived,OnCallCompleted).Integration Tests:
x-server-header,Set-Cookie) between client and server.End-to-End Tests:
Example Test Cases:
OnHeadersReceivedcorrectly captures server-sent headers.OnCallCompletedis triggered on both success and error cases.Checklist
Closes GitHub Issue: #138