-
Notifications
You must be signed in to change notification settings - Fork 191
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEATURE] Reimplement BulkProcessor #181
Comments
Would love it if someone (you?) could contribute this! I suppose that code can be just ported here? I don't see any obvious cons. |
FYI: We are currently looking into porting the While The RHLC allows to estimate the size of a Bulk Action (now called Can you think of an efficient way to perform such a size estimation or would you rather drop the |
I think that would significantly degrade the |
Elastic have just merged We did a quick (preliminary) port to the Would such a port be something that you'd consider worth and acceptable contributing? |
@ginkel this is very tricky (taking into account numerous precedents with Elastic as a company). Yes the license seems to be ASFv2, we may ask the contributor if he is open to submit the |
We will gladly accept code under the APLv2 license. The contributor needs to make sure that they have not looked at, or copied any non-APLv2 code while reimplementing a feature in OpenSearch. If the client is indeed APLv2 it's all good. |
All ported code carries the following license header:
|
@ginkel Yes. Keep those headers please and add the OpenSearch ones. |
Any updates on this? This is a blocker on https://issues.apache.org/jira/browse/NUTCH-2994. Let me know if I can help. |
Feel free to continue with the above strategy or any other solution that
works.
…On Thu, Jun 8, 2023 at 2:37 PM Tim Allison ***@***.***> wrote:
Any updates on this? This is a blocker on
https://issues.apache.org/jira/browse/NUTCH-2994. Let me know if I can
help.
—
Reply to this email directly, view it on GitHub
<#181 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA5PRLWT7LFG7SOG6EVG7T3XKILV3ANCNFSM54DZFVDA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Are there any plans on adding an equivalent of the BulkProcessor API, which was available in the High Level Rest Client, so that Index and Delete requests can be batched?
The text was updated successfully, but these errors were encountered: