ctubbsii commented on PR #3355: URL: https://github.com/apache/accumulo/pull/3355#issuecomment-1527772760
@keith-turner wrote: > Before this change it was calling the following code which could potentially add the ProblemReportingIterator and TimeSettingIterator. Right. I think it also limited the number of open readers (because there is a configurable max open) and caches the open readers for re-use. We're bypassing all of that, and I'm not sure if that's safe to do. It's very unlikely we're going to hit this code at all, given that nobody I know uses Mapfiles at all anymore. But I'm just not that familiar with the file reader code at the bottom of the iterator stack on the server side, and don't feel qualified to proceed with this if those things are important enough to not bypass. -- 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]
