-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
[Source] Snowflake - Add regex to getFailedTasksInDatabase #6736
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Looks good! Ready for QA!
Hello everyone, I have re-tested this PR and there's still a test cases failed. Please check the test report below for more information |
Closes #5867 |
…to feature/regex-snowflake-5867
@vunguyenhung Fixed. Also, I used this PR to make schema be an array |
Hi everyone, all test cases are passed! Ready for release! Test report |
WHAT
🤖 Generated by Copilot at f3c7be6
This pull request updates the snowflake package to version 0.9.1, which includes minor changes and bug fixes for several actions and sources related to snowflake. It also adds a new prop to the failed-task-in-schema source to filter failed tasks by name using regex, and refactors the common module of the snowflake sources to use helper methods for getting and setting the last max timestamp from the database. Additionally, it improves the performance and accuracy of the getFailedTasksInDatabase method of the snowflake app by using the INFORMATION_SCHEMA.TASK_HISTORY table function with some additional parameters.
🤖 Generated by Copilot at f3c7be6
WHY
HOW
🤖 Generated by Copilot at f3c7be6
failed-task-in-schema
source to allow filtering the failed tasks by a regex pattern (link)snowflake.app.mjs
file to accept the taskName parameter and use the INFORMATION_SCHEMA.TASK_HISTORY table function with additional parameters for performance and accuracy (link)common.mjs
module to abstract the logic of getting and setting the last max timestamp from the database (link)common.mjs
module to use the _getLastMaxTimestamp and _setLastMaxTimestamp methods instead of directly accessing the database, and to pass the taskName parameter to the getFailedTasksInDatabase method (link, link, link)