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.

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

Split job into model and index classes #3310

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

SugaryLump
Copy link
Contributor

  • The Job class has been split into [Job.java](http://job.java/) and [IndexedJob.java](http://indexedjob.java/), to work like entities in the code base that are split between a model class and an indexed class
  • A job's source objects are no longer indexed
  • Instances of Job usage in the codebase were replaced or overloaded where relevant to use either IndexedJob or more specific properties of a job
  • Any instance in the codebase where an indexed job's source objects were consulted now has a model access to retrieve this data
    • As an exception, tables that list jobs NO LONGER load data for job source objects, and this data is no longer included when exporting these tables to CSV
  • A new API method getJobFromModel' was added at /api/v2/jobs/{id}` that retrieves a job from the model with permission checking
  • The API method getJobPluginInfo now receives a dedicated PluginInfoRequest with only the required details as a path parameter instead of receiving a full Job object

@SugaryLump SugaryLump marked this pull request as ready for review February 3, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant