scwhittle commented on issue #34749: URL: https://github.com/apache/beam/issues/34749#issuecomment-2854541611
> If I understand well, you propose to have a static cache within AvroCoder to cache a full AvroCoder, right? Yup! as well as getting rid of those unnecessary thread-locals for the reader/writer as even if we have the AvroCoder cache that will duplicate those per-thread. > Would you like me to open a PR to do that? Also, would you consider merging this PR first so it has a chance of being included in the next beam release, and in parallel open another issue to work on your proposal? I believe that we have about a month before the next beam release cut. How about keeping the existing PR and trying the new approach in a separate PR? I don't think it will be much more complicated than the current PR so hopefully won't take long to code up. If we start getting close to deadline we can just merge the existing PR. -- 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: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
