Skip to content
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

[Backport 2.x] Support batch ingestion in bulk API (#12457) (#13306) #13462

Merged
merged 3 commits into from
Apr 30, 2024

Commits on Apr 30, 2024

  1. Support batch ingestion in bulk API (opensearch-project#12457) (opens…

    …earch-project#13306)
    
    * [PoC][issues-12457] Support Batch Ingestion
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Rewrite batch interface and handle error and metrics
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Remove unnecessary change
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Revert some unnecessary test change
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Keep executeBulkRequest main logic untouched
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Add UT
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Add UT & yamlRest test, fix BulkRequest se/deserialization
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Add missing java docs
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Remove Writable from BatchIngestionOption
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Add more UTs
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Fix spotlesscheck
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Rename parameter name to batch_size
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Add more rest yaml tests & update rest spec
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Remove batch_ingestion_option and only use batch_size
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Throw invalid request exception for invalid batch_size
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Update server/src/main/java/org/opensearch/action/bulk/BulkRequest.java
    
    Co-authored-by: Andriy Redko <[email protected]>
    Signed-off-by: Liyun Xiu <[email protected]>
    
    * Remove version constant
    
    Signed-off-by: Liyun Xiu <[email protected]>
    
    ---------
    
    Signed-off-by: Liyun Xiu <[email protected]>
    Signed-off-by: Liyun Xiu <[email protected]>
    Co-authored-by: Andriy Redko <[email protected]>
    (cherry picked from commit 1219c56)
    chishui committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    0c1fdea View commit details
    Browse the repository at this point in the history
  2. Adjust changelog item position to trigger CI

    Signed-off-by: Liyun Xiu <[email protected]>
    chishui committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    d78b7c2 View commit details
    Browse the repository at this point in the history
  3. Merge branch '2.x' into backport/backport-13306-to-2.x

    Signed-off-by: Liyun Xiu <[email protected]>
    chishui authored Apr 30, 2024
    Configuration menu
    Copy the full SHA
    587aed7 View commit details
    Browse the repository at this point in the history