Delete tenant data #378

Merged
tcdlpds merged 1 commits from ashvini/device-mgt-core:RemoveDeviceOrg into master 10 months ago
Fixes: https://roadmap.entgra.net/issues/10175
tcdlpds requested changes 10 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 10 months ago
tcdlpds requested changes 10 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 10 months ago
ashvini force-pushed RemoveDeviceOrg from 9ff9a97f2a to 159f603605 10 months ago
tcdlpds merged commit ea542894cf into master 10 months ago

Reviewers

tcdlpds requested changes 10 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.