Skip to content

[v9.1] feat: removed JobDB JobParameters table - #8236

Merged
aldbr merged 1 commit into
DIRACGrid:integrationfrom
fstagni:91_removeJobParameters
Oct 22, 2026
Merged

aldbr merged 1 commit into
DIRACGrid:integrationfrom
fstagni:91_removeJobParameters

Conversation

@fstagni

@fstagni fstagni commented Jun 27, 2025 •

Copy link
Copy Markdown
Contributor

BEGINRELEASENOTES

*WMS
CHANGE: removed JobDB.JobParameters table

*deployment

USE JobDB;
DROP TABLE IF EXISTS JobParameters

ENDRELEASENOTES

@fstagni
fstagni force-pushed the 91_removeJobParameters branch from aa8cc13 to 61e1b12 Compare June 27, 2025 12:57
@fstagni
fstagni force-pushed the 91_removeJobParameters branch from 61e1b12 to e4de33b Compare March 5, 2026 13:43
@fstagni fstagni changed the title [9.1] feat: removed JobDB JobParameters table [for minor release] feat: removed JobDB JobParameters table Mar 5, 2026
@fstagni
fstagni force-pushed the 91_removeJobParameters branch 2 times, most recently from ed98f0b to da55a04 Compare March 5, 2026 16:58
@fstagni
fstagni force-pushed the 91_removeJobParameters branch from da55a04 to 16d924e Compare July 6, 2026 15:55
@fstagni
fstagni force-pushed the 91_removeJobParameters branch 10 times, most recently from affb29d to 6ff28a5 Compare September 8, 2026 09:30
@fstagni
fstagni marked this pull request as ready for review September 8, 2026 11:44
@aldbr

aldbr commented Sep 9, 2026 •

Copy link
Copy Markdown
Contributor

ddev

Please also update diracx
Would also need to solve that issue first: DIRACGrid/diracx#767

UPDATE:
Actually I think everything is fine, JobParametersDB exists in diracx but is not used at all, so my points above are irrelevant I think and one can just remove it from diracx.

@aldbr aldbr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, this command could be adapted I think:

# getJobParameters returns {jobID: {parName: value, ...}, ...}, with jobID being either an int (JobDB)
# or a str (JobParametersDB), so look it up both ways

Comment thread src/DIRAC/WorkloadManagementSystem/Agent/JobCleaningAgent.py Outdated
@fstagni
fstagni force-pushed the 91_removeJobParameters branch from 6ff28a5 to ca15dbd Compare September 9, 2026 15:52
@fstagni
fstagni force-pushed the 91_removeJobParameters branch from ca15dbd to 9fd1d12 Compare October 5, 2026 14:26
@fstagni
fstagni force-pushed the 91_removeJobParameters branch 2 times, most recently from 6be763a to 610498e Compare October 21, 2026 15:03
@fstagni fstagni closed this Oct 22, 2026
@fstagni fstagni reopened this Oct 22, 2026
@fstagni fstagni changed the title [for minor release] feat: removed JobDB JobParameters table [v9.1] feat: removed JobDB JobParameters table Oct 22, 2026
@aldbr
aldbr enabled auto-merge October 22, 2026 14:14
@aldbr
aldbr disabled auto-merge October 22, 2026 14:15
@aldbr
aldbr enabled auto-merge October 22, 2026 14:16
@aldbr
aldbr force-pushed the 91_removeJobParameters branch from 610498e to defe4a6 Compare October 22, 2026 14:17
@aldbr
aldbr merged commit c26cbfd into DIRACGrid:integration Oct 22, 2026
12 of 23 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Oct 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants