aws-samples / aws-genai-conversational-rag-reference

AWS Generative AI Conversational RAG Reference (Galileo)
https://aws-samples.github.io/aws-genai-conversational-rag-reference
Apache License 2.0
63 stars 20 forks source link

[BUG] (Reset Vector Data Does not work) #107

Closed nghiapt closed 1 month ago

nghiapt commented 4 months ago

Describe the bug

The path to reset vector store data (https://aws-samples.github.io/aws-genai-conversational-rag-reference/development/vector-store/#how-to-reset-vector-store-data) does not work as DynamoDB permission for writing tables is missing, and the TRUNCATE query for Postgresql has an invalid syntax.

Expected Behavior

The Step function execution works successfully

Current Behavior

Step Function execution failed at ConfigTask (DynamoDB IAM error) and VectorStoreSetupTasK (Postgresql syntax error)

Reproduction Steps

{
  "VectorStoreManagement": {
    "PurgeData": true,
    "CreateIndexes": false
  }
}

Possible Solution

Add DynamoDB IAM error and use interpolation instead of parameterized query

Additional Information/Context

No response

Environment details (OS name and version, etc.)

system: Linux 6.2.0-1018-aws #18~22.04.1-Ubuntu SMP Wed Jan 10 22:54:16 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux pnpm: 8.12.1 node: v20.10.0 python: Python 3.10.12 poetry: Poetry (version 1.7.1) docker: Docker version 24.0.7, build afdd53b java: openjdk 18.0.2 2022-07-19 OpenJDK Runtime Environment Corretto-18.0.2.9.1 (build 18.0.2+9-FR) OpenJDK 64-Bit Server VM Corretto-18.0.2.9.1 (build 18.0.2+9-FR, mixed mode, sharing) aws: aws-cli/2.15.2 Python/3.11.6 Linux/6.2.0-1018-aws exe/x86_64.ubuntu.22 prompt/off

github-actions[bot] commented 2 months ago

This issue is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label.

github-actions[bot] commented 1 month ago

Closing this issue as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. If you wish to exclude this issue from being marked as stale, add the "backlog" label.