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

[FEATURE] Support batch ingestion in TextEmbeddingProcessor & SparseEncodingProcessor #743

Closed
chishui opened this issue May 9, 2024 · 11 comments
Assignees

Comments

@chishui
Copy link
Contributor

chishui commented May 9, 2024

Is your feature request related to a problem?

RFC: opensearch-project/OpenSearch#12457

We have implemented batch ingestion logic in OpenSearch core in version 2.14, now we want to enable the batch ingestion capability in neural-search processors: TextEmbeddingProcessor & SparseEncodingProcessor so that we can better utilize the remote ML server's GPU capacity and accelerate the ingestion process, based our benchmark, batch can reduce total ingestion time by 77% without seeing throttling error (P90, SageMaker), please refer to here to see the benchmark results.

What solution would you like?

  1. In InferenceProcessor, override Processor's batchExecute API, add a default implementation to combine List<String> inferenceText from multiple docs, then reuse the mlCommonsClientAccessor.inferenceSentences and mlCommonsClientAccessor.inferenceSentencesWithMapResult. After getting inference results, map them to each doc and update the docs.
  2. We'll sort the docs by length before sending for inference to achieve better performance. And inference results will be restored to original order before being processed.
    (This was original proposed in ml-commons. But as @ylwu-amzn suggested that we can reuse input_docs_processed_step_size as max batch size, then it makes more sense to sort the docs in neural-search as we can ensure that we won't sort docs from TextImageEmbeddingProcessor)

What alternatives have you considered?

N/A

Do you have any additional context?

N/A

@zhichao-aws
Copy link
Member

Hi @chishui , could you please provide an example API request body to create a batched ingest processor?

@martin-gaievski
Copy link
Member

We are not adding this feature for TextImageEmbeddingProcessor, is there a plan to do it later?

@chishui
Copy link
Contributor Author

chishui commented May 20, 2024

@zhichao-aws there is no changes to how TextEmbeddingProcessor and SparseEncodingProcessor are created. It's only when user uses _bulk API with batch_size parameter, then the processors will see documents in batches.

@chishui
Copy link
Contributor Author

chishui commented May 21, 2024

@martin-gaievski we don't have a plan to support TextImageEmbeddingProcessor as it requires text and image to be grouped in one request.

@chishui
Copy link
Contributor Author

chishui commented May 21, 2024

Updated description in "What solution would you like?" section.

@chishui chishui closed this as completed May 28, 2024
@navneet1v
Copy link
Collaborator

@chishui given that the feature is merged across all the components, what is the final benchmarking number for the batch ingestion?

@navneet1v
Copy link
Collaborator

@chishui can you attach the documentation issue here for tracking purpose as this feature.

@navneet1v
Copy link
Collaborator

@chishui I looked into the benchmarks here: opensearch-project/OpenSearch#12457 (comment), when we say there is a significant improvement on throughput I see it is with a gpu instance and not with any other model. Is this correct?

@chishui
Copy link
Contributor Author

chishui commented May 30, 2024

@chishui given that the feature is merged across all the components, what is the final benchmarking number for the batch ingestion?

This is the recent benchmark I redid with neural sparse model hosted on sagemaker

SageMaker

  • SageMaker host type: g5.xlarge
  • Processor: Sparse Encoding
  • Benchmark Setup
    • Bulk size: 100
    • client: 1
Metrics no batch batch (batch size=50, input_docs_processed_step_size =20)
Min Throughput (docs/s) 57.73 17.35
Mean Throughput (docs/s) 93.26 214.94
Median Throughput (docs/s) 93.9 226.36
Max Throughput (docs/s) 96.38 239.91
Latency P50 (ms) 1180.89 427.71
Latency P90 (ms) 1134.32 562.268
Latency P99 (ms) 1248.18 796.35
Total Benchmark Time (s) 3275 1361
Error Rate (%) 0 0

@chishui
Copy link
Contributor Author

chishui commented May 30, 2024

@chishui I looked into the benchmarks here: opensearch-project/OpenSearch#12457 (comment), when we say there is a significant improvement on throughput I see it is with a gpu instance and not with any other model. Is this correct?

We did benchmarked on Sagemaker, OpenAI and Cohere which all results were listed in the comment you linked. We saw improvement on throughput for all these services. The gpu instance mentioned was the one SageMaker used as we can choose instance type for SageMaker but we have no idea what type of gpu instance OpenAI and Cohere used.

@chishui
Copy link
Contributor Author

chishui commented Jun 6, 2024

@chishui can you attach the documentation issue here for tracking purpose as this feature.

opensearch-project/documentation-website#7305

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants