Skip to content

CB-3616 add logs for local rm#1839

Merged
alexander-skoblikov merged 5 commits intodevelfrom
CB-3616-extra-logging-for-rm
Jul 27, 2023
Merged

CB-3616 add logs for local rm#1839
alexander-skoblikov merged 5 commits intodevelfrom
CB-3616-extra-logging-for-rm

Conversation

@yagudin10
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Collaborator

@alexander-skoblikov alexander-skoblikov left a comment

Choose a reason for hiding this comment

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

Add information about who change data

BaseWebProjectImpl webProject = getWebProject(projectId, false);
doFileWriteOperation(projectId, webProject.getMetadataFilePath(),
() -> {
log.debug("Updating value for property '" + propName + "'");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Provide more info (at least project ID)

DBPDataSourceContainer dataSource = registry.getDataSource(dataSourceId);

if (dataSource != null) {
log.debug("Deleting data source '" + dataSourceId + "'");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also add project info here and all project-related functions

@yagudin10 yagudin10 requested a review from serge-rider July 24, 2023 15:32
@serge-rider serge-rider marked this pull request as ready for review July 26, 2023 08:19
@alexander-skoblikov alexander-skoblikov merged commit 07fc1f8 into devel Jul 27, 2023
@serge-rider serge-rider deleted the CB-3616-extra-logging-for-rm branch August 21, 2023 13:26
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.

4 participants