Fixes and improvements for Dynamic & random schedule tasks #350

Merged
tcdlpds merged 3 commits from charithag/device-mgt-core:task-fixes into master 9 months ago
Owner

Changes proposed in this PR required to rune below sql query for migration.

ALTER TABLE DYNAMIC_TASK MODIFY COLUMN CRON VARCHAR(100);
ALTER TABLE DYNAMIC_TASK MODIFY COLUMN TASK_CLASS_NAME VARCHAR(1000);
ALTER TABLE DYNAMIC_TASK_PROPERTIES MODIFY COLUMN PROPERTY_VALUE TEXT;
ALTER TABLE DYNAMIC_TASK_PROPERTIES MODIFY COLUMN TENANT_ID INTEGER DEFAULT 0;
Changes proposed in this PR required to rune below sql query for migration. ```sql ALTER TABLE DYNAMIC_TASK MODIFY COLUMN CRON VARCHAR(100); ALTER TABLE DYNAMIC_TASK MODIFY COLUMN TASK_CLASS_NAME VARCHAR(1000); ALTER TABLE DYNAMIC_TASK_PROPERTIES MODIFY COLUMN PROPERTY_VALUE TEXT; ALTER TABLE DYNAMIC_TASK_PROPERTIES MODIFY COLUMN TENANT_ID INTEGER DEFAULT 0; ```
amalka.subasinghe was assigned by charithag 9 months ago
pahansith was assigned by charithag 9 months ago
charithag added 2 commits 9 months ago
tcdlpds was assigned by charithag 9 months ago
tcdlpds requested changes 9 months ago
operationTimeoutConfig.getInitialStatus());
for (Activity activity : activities) {
for (ActivityStatus activityStatus : activity.getActivityStatus()) {
String operationId = activity.getActivityId().replace("ACTIVITY_", "");
Owner

Isn't it possible to move line number 94 to outside of the inner loop? Since it is getting from activity, I think it is possible.

Isn't it possible to move line number 94 to outside of the inner loop? Since it is getting from activity, I think it is possible.
Poster
Owner

done

done
charithag marked this conversation as resolved
charithag force-pushed task-fixes from 32ac69872f to d2ad8ac579 9 months ago
tcdlpds merged commit 09772dfd3f into master 9 months ago

Reviewers

tcdlpds requested changes 9 months ago
The pull request has been merged as 09772dfd3f.
Sign in to join this conversation.
No reviewers
No Milestone
No project
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: community/device-mgt-core#350
Loading…
There is no content yet.