nddipiazza commented on PR #2488: URL: https://github.com/apache/tika/pull/2488#issuecomment-3694844408
## 🔧 Critical Fix Added Fixed a bug where dynamically created fetchers via gRPC were not available in the forked PipesServer process. ### The Problem - `saveFetcher` was only saving to the gRPC server's local FetcherManager - When `pipesClient.process()` forks a new PipesServer, that process has its own FetcherManager - Result: `FetcherNotFoundException: Can't find fetcher for id=defaultFetcher` ### The Solution - `saveFetcher` now calls **both** `fetcherManager.saveFetcher()` AND `pipesClient.saveFetcher()` - This propagates the fetcher to the forked PipesServer via the socket protocol - Also implemented `deleteFetcher` properly (was previously a no-op) ### Testing The e2e test should now pass - fetchers created via gRPC will be available for document processing. -- 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]
