-
Notifications
You must be signed in to change notification settings - Fork 48
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
[bugfix] Replace CronUtils
library
#196
base: main
Are you sure you want to change the base?
[bugfix] Replace CronUtils
library
#196
Conversation
CronUtils
libraryCronUtils
library
hi @pariksheet , @asingamaneni , need your opinion.
Do you think we need to aim for the full compatibility with Quartz, or whatever is there is enough? |
I don't think we would need any more than what you have mentioned in the examples. I am not sure if anyone is using L-W in their pipelines. We could ask in the support channel to confirm! |
Description
As stated in the #195, using CronUtils library on DBR 15.4 is impossible due to lack of access to JVM. This library is available for Java only. Hence I've implemented custom code to convert between unix cron and quartz cron and removed the dependency on the said library.
Currently not all features of the Quartz expression are supported, namely: L W and # keys.
Related Issue
#195
Motivation and Context
Brickflow can't be used with DBR 15.4 and future versions as long as plugins are required.
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: