Delete tenant data #378

Merged
tcdlpds merged 1 commits from ashvini/device-mgt-core:RemoveDeviceOrg into master 8 months ago
Fixes: https://roadmap.entgra.net/issues/10175
tcdlpds requested changes 8 months ago
} catch (Exception e) {
String msg = "Error getting tenant ID from domain: "
+ tenantDomain;
log.error(msg);
Owner

Log exception as well.

Log exception as well.
log.error(msg, e);
throw new ApplicationManagementException(msg, e);
} catch (ApplicationManagementDAOException e) {
String msg = "Database access error is occurred when getting applications for tenant with ID: " + tenantId;
Owner

Don't we need to rollback here?

Don't we need to rollback here?
log.error(msg, e);
throw new ApplicationManagementException(msg, e);
} catch (LifeCycleManagementDAOException e) {
String msg = "Error occurred while deleting life-cycle state data of application releases of the tenant"
Owner

Don't we need to rollback here?

Don't we need to rollback here?
log.error(msg, e);
throw new ApplicationManagementException(msg, e);
} catch (ReviewManagementDAOException e) {
String msg = "Error occurred while deleting reviews of application releases of the applications"
Owner

Don't we need to rollback here?

Don't we need to rollback here?
import com.google.common.reflect.TypeToken;
import com.google.gson.Gson;
import io.entgra.device.mgt.core.device.mgt.common.metadata.mgt.DeviceStatusManagementService;
import io.entgra.device.mgt.core.device.mgt.core.dao.*;
Owner

Do not use wild card imports

Do not use wild card imports
ashvini force-pushed RemoveDeviceOrg from cf57b77ce0 to 655e979e6c 8 months ago
tcdlpds requested changes 8 months ago
}
try {
DeviceManagementDAOFactory.openConnection();
Owner

Here you have to begin a transaction.

Here you have to begin a transaction.
ashvini force-pushed RemoveDeviceOrg from 655e979e6c to 858cbd9511 8 months ago
ashvini force-pushed RemoveDeviceOrg from 9ff9a97f2a to 159f603605 8 months ago
tcdlpds merged commit ea542894cf into master 8 months ago

Reviewers

tcdlpds requested changes 8 months ago
The pull request has been merged as ea542894cf.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

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