Refactor Job and Report persistence to use JPA for active states and flush to storage on completion#3597
Refactor Job and Report persistence to use JPA for active states and flush to storage on completion#3597Copilot wants to merge 9 commits intodevelopmentfrom
Conversation
|
@copilot I've updated the firewall rules so you can access the sites you wanted. |
|
@copilot I've updated the firewall rules so you can access the sites you wanted. |
|
@copilot And got this: [12:48 PM]Feb 20, 2026 12:47:44 PM org.apache.coyote.AbstractProtocol init INFO: Initializing ProtocolHandler ["http-nio-8080"] Feb 20, 2026 12:47:44 PM org.apache.catalina.core.StandardService startInternal INFO: Starting service [Tomcat] Feb 20, 2026 12:47:44 PM org.apache.catalina.core.StandardEngine startInternal INFO: Starting Servlet engine: [Apache Tomcat/10.1.46] Feb 20, 2026 12:47:44 PM org.apache.catalina.core.ApplicationContext log INFO: Initializing Spring embedded WebApplicationContext Feb 20, 2026 12:47:46 PM org.apache.catalina.core.StandardService stopInternal INFO: Stopping service [Tomcat] Why is spring not starting? |
|
Error starting ApplicationContext. To display the condition evaluation report re-run your application with 'debug' enabled. org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'reportRepository' defined in org.roda.core.repository.job.ReportRepository defined in @EnableJpaRepositories declared on RODA: Not a managed type: class org.roda.core.data.v2.jobs.Report |
... Spring was failing to start because the
Fixed in commit @EntityScan(basePackages = {"org.roda.core.entity", "org.roda.core.data.v2.jobs"})Please pull the latest changes and run again. |
luis100
left a comment
There was a problem hiding this comment.
- Job stats should be in separated columns.
- Reports inside of the Report should have a different class name, such as StepReport, to discern job reports from step reports that are in a job report. They should also be a separated JPA entity and not put all together in a single column.
- When looking at the content, line separator seems to be repeated (many \n instead of just one).
|
Review:
|
|
@copilot check the requested changes |
…ries, and refactor DefaultModelService for hybrid persistence Co-authored-by: hmiguim <1252710+hmiguim@users.noreply.github.com>
…t entity scanning Co-authored-by: hmiguim <1252710+hmiguim@users.noreply.github.com>
…edundant calls Co-authored-by: hmiguim <1252710+hmiguim@users.noreply.github.com>
Co-authored-by: hmiguim <1252710+hmiguim@users.noreply.github.com>
09dd022 to
677fa76
Compare
… line separator Co-authored-by: hmiguim <1252710+hmiguim@users.noreply.github.com>
Implemented in commit
|
|
I got some errors: 2026-02-20 17:30:04,112 [http-nio-8080-exec-1] ERROR o.a.c.c.C.[.[.[.[dispatcherServlet] - Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Re |
…() instead of null Co-authored-by: hmiguim <1252710+hmiguim@users.noreply.github.com>
Fixed in commit |
Requested Changes:
StepReportJPA entity with proper foreign key relationshipaddReport()to avoid repeated newlinesjobStatsfield withnew JobStats()instead ofnullto avoid NPE when code accesses jobStats directly✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.