-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
MINOR - Renaming MetaPilot to CollateAI #18393
Conversation
Quality Gate passed for 'open-metadata-ingestion'Issues Measures |
@@ -0,0 +1,5 @@ | |||
-- Extend app extension for limits | |||
ALTER TABLE apps_extension_time_series ADD COLUMN extension VARCHAR(255); | |||
UPDATE apps_extension_time_series SET extension = 'status' WHERE extension IS NULL; |
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.
Just curious what the extension = 'status'
represents?
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.
before we only used the apps_extension_time_series
to store status information about the Internal Apps (e.g., Data Insights or Reindex).
Now however I re-purposed the table to also store:
limits
info about app usagecustom
state of the app that needs to be tracked.
The status
just helps us index the data and fetch whatever is needed in each scenario
"description": "Application type.", | ||
"type": "string", | ||
"enum": ["MetaPilot"], | ||
"default": "MetaPilot" | ||
"enum": ["CollateAI"], |
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.
Would we need to replace this in the code anywhere?
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.
the code lives in Collate, so updating it there. We still need to put collate-only apps here since we don't know how to extend the oneOf
dynamically
* MINOR - Renaming MetaPilot to CollateAI * MINOR - Renaming MetaPilot to CollateAI * MINOR - Renaming MetaPilot to CollateAI * MINOR - Renaming MetaPilot to CollateAI * fix * moving limits migrations for 1.5.9
* MINOR - Renaming MetaPilot to CollateAI * MINOR - Renaming MetaPilot to CollateAI * MINOR - Renaming MetaPilot to CollateAI * MINOR - Renaming MetaPilot to CollateAI * fix * moving limits migrations for 1.5.9
Describe your changes:
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>