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

feat: PI Disaggregation objects [DHIS2-18745] #19601

Draft
wants to merge 2 commits into
base: DHIS2-18745
Choose a base branch
from

Conversation

jimgrace
Copy link
Member

@jimgrace jimgrace commented Jan 7, 2025

Creating objects for PI Disaggregation -- work in pogress

@jimgrace jimgrace marked this pull request as draft January 7, 2025 00:17
public class ProgramCategoryOptionMapping implements EmbeddedObject {

/** The database internal identifier for this Object. */
private int id;
Copy link
Contributor

Choose a reason for hiding this comment

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

The id column is mapped as bigint in database so I think it should be long java type.

@@ -76,7 +76,7 @@ public class JobScheduler implements Runnable, JobRunner {
* <li>Leaves as much time as possible otherwise to get through the work of triggering jobs
* </ul>
*/
private static final int LOOP_SECONDS = 20;
private static final int LOOP_SECONDS = 20000000;
Copy link
Contributor

Choose a reason for hiding this comment

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

???

Copy link
Member Author

Choose a reason for hiding this comment

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

@jbee Ha, that's temporary! When I'm debugging I sometimes find it very helpful to watch my Postgres log in real time to see what's happening in the database. (I log all statements all the time and rotate the log by day of week so it doesn't fill up my disk.) When I don't need the job scheduler (which is nearly all the time), I change this value so it's not dumping in a bunch of queries three times a minute. It make the log much easier to follow!

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.

3 participants