Cleanup compile-time warnings introduced through recent additions

jakartaee / persistence

https://jakartaee.github.io/persistence/

Other

207 stars 59 forks source link

Cleanup compile-time warnings introduced through recent additions #481

Closed lukasj closed 1 year ago

lukasj commented 1 year ago

ie PersistenceConfiguration is missing a constructor - ie do we want to have just default public one and/or rather a specific one taking some arguments?

gavinking commented 1 year ago

@lukasj please review #482 which I think addresses your concerns and more.