enragedginger/akka-quartz-scheduler

Make QuartzSchedulerExtension#removeSchedule public

nebehr opened this issue · 3 comments

nebehr commented

Since createSchedule method is public it makes sense to make removeSchedule public as well. Currently a schedule can be deleted by removing entry from schedules map but ideally IT should be made private instead.

https://github.com/enragedginger/akka-quartz-scheduler/blob/master/src/main/scala/QuartzSchedulerExtension.scala#L241

enragedginger commented

@nebehr Thanks for the suggestion. I can see the value of this. It's a bit more complicated than simply removing the schedule as you'd want to cancel the associated jobs first. Feel free to submit a PR for this.

vendethiel commented

Why close the issue? Even if you're waiting for someone to contribute the feature, it's actually useful to have that issue open.

enragedginger commented

@vendethiel I'm not waiting for someone to contribute the feature at this point. If someone wants to volunteer to do it and submit the PR, they're welcome to do it. I prefer to use issues on GitHub projects for true issues with the software or possibly active feature development. The "projects" section of each project exists specifically to track future work.