Skip to content

Conversation

@sshekhar1
Copy link
Contributor

@sshekhar1 sshekhar1 commented Sep 1, 2025

Below are the screenshots for the working demo:

  • Selecting Vector Store Provider under Upsert Chunks option:
image
  • Using Teradata Vector Store under Upsert Chunks option in Document Store:
image
  • Upsert successful into Teradata Vector Store:
image
  • Test Retrieval on Teradata Vector Store:
image
  • Teradata Vector Store node under ChatFlow:
image_721

@HenryHengZJ
Copy link
Contributor

are there any screenshots for working demo or recordings?

@sshekhar1 sshekhar1 force-pushed the Teradata_DocumentStore branch from 6729347 to 79620b4 Compare September 8, 2025 07:09
@sshekhar1
Copy link
Contributor Author

are there any screenshots for working demo or recordings?

@HenryHengZJ I have added the screenshots in the Pull Request Description as suggested. Thanks !

@HenryHengZJ
Copy link
Contributor

can you do pnpm lint-fix?

@sshekhar1
Copy link
Contributor Author

Output for pnpm lint-fix:

$ pnpm lint-fix

> [email protected] lint-fix C:\Users\ss186247\OneDrive - Teradata Corporation\Documents\FlowiseAI\flowise_repo_build\Flowise
> pnpm lint --fix


> [email protected] lint C:\Users\ss186247\OneDrive - Teradata Corporation\Documents\FlowiseAI\flowise_repo_build\Flowise
> eslint "**/*.{js,jsx,ts,tsx,json,md}" "--fix"

@sshekhar1
Copy link
Contributor Author

@HenryHengZJ can you please help review the changes and let me know if any additional changes needed ?

constructor() {
this.label = 'Teradata Vector Store API Credentials'
this.name = 'teradataVectorStoreApiCredentials'
this.version = 1.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we add description on how to get these credentials? as if user is new to teradata

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or perhaps we can create a docs page on https://github.com/FlowiseAI/FlowiseDocs and link to that

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The credentials related info is same as that mentioned in the below pull request:
#5002

@HenryHengZJ
Copy link
Contributor

thank you!

@HenryHengZJ HenryHengZJ merged commit ac565b8 into FlowiseAI:main Oct 28, 2025
2 checks passed
@HenryHengZJ
Copy link
Contributor

this is exactly what I was worrying about:
image

Seems like teradatasql does not support arm64 build, hence the error when building Docker image

@sshekhar1
Copy link
Contributor Author

this is exactly what I was worrying about: image

Seems like teradatasql does not support arm64 build, hence the error when building Docker image

@HenryHengZJ The error doesn't seem to suggest that teradatasql doesn't support arm64 build. It seems to be related to missing build dependencies in the Docker container. The error CMake does not seem to be available indicates the container lacks the tools needed to compile native modules from source. It may require installing missing tools/dependencies for successful compilation.

The possible solution to this could be to use multi-stage build, where in "Build Stage" the build tools/dependencies can be installed for native module compilation and in "Production Stage" compiled node_modules can be copied from builder stage. Could you please check if this helps ?

@sshekhar1
Copy link
Contributor Author

sshekhar1 commented Nov 19, 2025

@HenryHengZJ Hope you are doing fine. Did you get a chance to look into the build issue resolution somehow ? Please check my last comment and let me know if it can help fix build issue or am I missing something here.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants