enragedginger / akka-quartz-scheduler

Quartz Extension and utilities for cron-style scheduling in Akka
Other
559 stars 114 forks source link

User friendly log message on job setup #110

Open pawelkaczor opened 3 years ago

pawelkaczor commented 3 years ago

Fix #93

pawelkaczor commented 3 years ago

The changes have been already discussed (see #94) and rejected due to incorporation of new dependencies. I'm re-opening the PR in response to comment https://github.com/enragedginger/akka-quartz-scheduler/issues/93#issuecomment-841095711 by @felipebonezi

New dependencies incorporated by this PR:

pawelkaczor commented 3 years ago

Perhaps we should introduce new, optional module that would provide desired functionality.

felipebonezi commented 3 years ago

I still think we should take a different approach here. See my rationale here: #93 (comment)

Is there any way to create an plugin that users choose to attach or not? So, if developer wants to use the cron parser, than he must accept the libraries that goes with it.

pawelkaczor commented 3 years ago

@felipebonezi I switched to https://github.com/eikek/calev#akka . Perhaps you will like it as well.