-
Notifications
You must be signed in to change notification settings - Fork 28
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
[new feature] Implement Assets - part 1 #359
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
AssetPackage assetPackage = assetManagerPackageLoader.loadPackageForAsset(assetType); | ||
|
||
if (assetPackage != null) { | ||
log.info("Found the package the agent belongs to: {}", assetPackage.getName()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
log.info("Found the package the agent belongs to: {}", assetPackage.getName()); | |
log.info("Found the package the asset belongs to: {}", assetPackage.getName()); |
throw new RuntimeException( | ||
"Package " | ||
+ assetPackage.getName() | ||
+ " declared to support agent type " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+ " declared to support agent type " | |
+ " declared to support asset type " |
+ assetPackage.getName() | ||
+ " declared to support agent type " | ||
+ assetType | ||
+ " but no agent found"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+ " but no agent found"); | |
+ " but no asset found"); |
return agentCodeProviderProvider; | ||
} | ||
|
||
log.info("No agent found in the package, let's try to find it among all the packages"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
log.info("No agent found in the package, let's try to find it among all the packages"); | |
log.info("No asset found in the package, let's try to find it among all the packages"); |
} | ||
} | ||
|
||
throw new RuntimeException("No AgentCodeProvider found for type " + assetType); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
throw new RuntimeException("No AgentCodeProvider found for type " + assetType); | |
throw new RuntimeException("No AssetPackage found for type " + assetType); |
thank you @nicoloboschi for your review |
Fixes #353
Summary:
The main use case here is to create tables of vector databases in order to fully set up the application when you deploy it
This is a preliminary PR that builds all the infrastructure to make Assets work.
The next step is to implement the "cassandra-table" AssetManager (and maybe validation at the planner side of the available asset types)