rzo1 commented on issue #2212: URL: https://github.com/apache/stormcrawler/issues/2212#issuecomment-6165428778
+1, as long as the code moves into the existing `opensearch` module and isn't renamed. Then it replaces the old module in place: same artifact, same packages and class names, same config keys, and only one OpenSearch module afterwards. Users shouldn't get confused. Before we switch, I'd like to see: - 429 retries with backoff and flushing by size. Using `BulkIngester` seems the easier option, since it does both already. - A short migration note in the PR description for `sniff` and `responseBufferSize`, which are no longer supported, plus the other differences dpol1 listed. -- 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]
