alamb commented on PR #8436: URL: https://github.com/apache/arrow-rs/pull/8436#issuecomment-3339042900
> > > Ouch time 😅 > > > > > > As in this caused merge conflicts with the thrift-remodel branch? > > Precisely 😁 > > Took me two tries, but I got it merged. But that's what I signed up for. Actually I've been amazed so far by the lack of conflicts. Ideally the code to parse thrift should be pretty isolated (though I realize that is not the current state of main 😆 ) I feel like we are finally getting to the point where parsing / decoding and IO are all separated nicely, which I think sets us up very nicely for additional performance improvements -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
