From a598e5ea0badf584af698f3b5c3b1ca61db76cf3 Mon Sep 17 00:00:00 2001
From: Austin Lee <austin@aryn.ai>
Date: Wed, 11 Oct 2023 16:10:40 -0700
Subject: [PATCH] Address review comments.

Signed-off-by: Austin Lee <austin@aryn.ai>
---
 search-processors/README.md                                     | 2 ++
 .../searchpipelines/questionanswering/generative/llm/Llm.java   | 1 +
 2 files changed, 3 insertions(+)

diff --git a/search-processors/README.md b/search-processors/README.md
index 7bb8572b41..030691855b 100644
--- a/search-processors/README.md
+++ b/search-processors/README.md
@@ -51,6 +51,8 @@ GET /<index>/_search\?search_pipeline\=<search pipeline name>
 
 To use this with Bedrock models, use "bedrock/" as a prefix for the "llm_model" parameters, e.g. "bedrock/anthropic".
 
+The latest RAG processor has been tested with OpenAI's GPT 3.5 and 4 models and Bedrock's Anthropic Claude (v2) model only.
+
 ## Retrieval Augmented Generation response
 ```
 {
diff --git a/search-processors/src/main/java/org/opensearch/searchpipelines/questionanswering/generative/llm/Llm.java b/search-processors/src/main/java/org/opensearch/searchpipelines/questionanswering/generative/llm/Llm.java
index faf136d550..be5efdb294 100644
--- a/search-processors/src/main/java/org/opensearch/searchpipelines/questionanswering/generative/llm/Llm.java
+++ b/search-processors/src/main/java/org/opensearch/searchpipelines/questionanswering/generative/llm/Llm.java
@@ -22,6 +22,7 @@
  */
 public interface Llm {
 
+    // TODO Ensure the current implementation works with all models supported by Bedrock.
     enum ModelProvider {
         OPENAI,
         BEDROCK