Skip to content
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.

Sign up for GitHub

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

Jump to bottom

No setter for property 'discardIdleConnectionsSeconds' #264

Closed
lcardito opened this issue Oct 3, 2018 · 3 comments
Closed

No setter for property 'discardIdleConnectionsSeconds' #264

lcardito opened this issue Oct 3, 2018 · 3 comments
Assignees
@zemian

Comments

@lcardito
Copy link

lcardito commented Oct 3, 2018

Hey guys,

when defining the property

org.quartz.dataSource.NAME.discardIdleConnectionsSeconds = 60

as per documentation ConfigDataSources, we're getting the following error:

Caused by: org.quartz.SchedulerException: Could not initialize DataSource: schedule
	at org.quartz.impl.StdSchedulerFactory.instantiate(StdSchedulerFactory.java:1042)
	at org.quartz.impl.StdSchedulerFactory.getScheduler(StdSchedulerFactory.java:1559)
	... 16 common frames omitted

Looks like the issue has been reported a while back on JIRA with the workaround:

org.quartz.dataSource.NAME.maxIdleTime = 60

Should the documentation be updated perhaps?

Thanks!
@zemian
Copy link
Contributor

zemian commented Dec 18, 2018

Can you share your quartz.properties that has this problem? I think only Hikari Pool supports this property.

@zemian
Copy link
Contributor

zemian commented Dec 29, 2018

This problem is very similar to #281. We can make additional changes to similar fix to handle this property as well.

@zemian zemian self-assigned this Dec 29, 2018
zemian added a commit that referenced this issue Dec 29, 2018
@zemian
#264: Fix no setter for dataSource property discardIdleConnectionsSec…
a91a1d0 
…onds
@zemian
Copy link
Contributor

zemian commented Dec 29, 2018

Resolved ece2753

@zemian zemian closed this as completed Dec 29, 2018
This was referenced Mar 8, 2021
Open
Merged
Open
Closed
This was referenced Mar 16, 2021
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zemian zemian

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lcardito @zemian