pjfanning commented on PR #2924: URL: https://github.com/apache/pekko/pull/2924#issuecomment-4412312909
Having an extra, 4th, ByteString impl comes with too large a maintenance overhead. Lots more code and lots more tests and easy to miss having test coverage. I don't think there are too many places where we benefit - instances where we concat exactly 2 byte strings that we don't later concat or that have been previously concatenated. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
