andrewglowacki commented on issue #2331: URL: https://github.com/apache/accumulo/issues/2331#issuecomment-955708431
After looking main over more closely, I see what @dlmarion was talking about with regard to changes in #1840. I decided not to do anything on main since #1840 explicitly prevented TabletServerBatchWriter from catching Throwbles. However, after considering the discussion in #1840, I think the changes there were all made from the server's perspective even though TabletServerBatchWriter is primarily a client API. It seems TabletServerBatchWriter (and possibly other classes used by the client) should be more cognizant of the environment it is running in when deciding whether to report Throwables to the client verses halting the JVM. I don't think libraries should be making these kinds of decisions for their user. -- 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]
