Skip to content

Allow job stream without database#34

Merged
iamneal merged 3 commits intomasterfrom
allow-job-stream-without-database
Jan 20, 2026
Merged

Allow job stream without database#34
iamneal merged 3 commits intomasterfrom
allow-job-stream-without-database

Conversation

@bdodroid
Copy link
Contributor

This pull request updates the GateClientJobStream to improve how jobs are handled when the plugin is not running, particularly for admin api requests. The main enhancement is that non-admin jobs are now explicitly rejected with an error message if the database is unavailable, and this error is reported back to the server. Several helper methods and imports were also added to support this functionality.

Job handling improvements:

  • Non-admin jobs are now rejected with a clear error message if the plugin is not running (i.e., when the database is unavailable and the system is in admin-only mode). The error is submitted back to the server using the new submitJobError method. [1] [2] [3]
  • A helper method isAdminJob was introduced to identify admin-only jobs, allowing them to bypass the plugin running check.

Codebase enhancements:

  • Added the SubmitJobResultsRequest import and a new submitJobError method to encapsulate error submission logic. [1] [2]
  • Introduced a DEFAULT_TIMEOUT_SECONDS constant and imported TimeUnit to standardize gRPC request timeouts. [1] [2]
  • Simplified the plugin running check in the start() method to only check for configuration, aligning with the new job handling logic.

Braden Olsen added 3 commits January 20, 2026 09:22
Implements logic to skip non-admin jobs when the database is unavailable,
ensuring only admin jobs proceed in admin-only mode. Introduces a method
for submitting job errors via gRPC service with a default timeout setting.
Adjusts job start conditions to streamline initialization process.

Relates to branch: allow-job-stream-without-database
its unimplemented currently
@bdodroid bdodroid requested a review from iamneal January 20, 2026 17:33
@iamneal iamneal merged commit b8d3682 into master Jan 20, 2026
3 checks passed
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.

2 participants