Skip to content

870: get rid of cluttering logs#870

Merged
alfardil merged 1 commit intomainfrom
870
Mar 19, 2026
Merged

870: get rid of cluttering logs#870
alfardil merged 1 commit intomainfrom
870

Conversation

@alfardil
Copy link
Collaborator

@alfardil alfardil commented Mar 19, 2026

870

Description of changes

Removed wasteful logs

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

They no longer show up.

Staging

@github-actions
Copy link
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@alfardil
Copy link
Collaborator Author

/ai

@tahminator
Copy link
Owner

/review

@tahminator
Copy link
Owner

/describe

@tahminator
Copy link
Owner

/improve

@github-actions
Copy link
Contributor

Title

870: get rid of cluttering logs


PR Type

Enhancement


Description

  • Removed verbose info logs.

  • Streamlined submission scheduler output.

  • Improved log clarity and relevance.


File Walkthrough

Relevant files
Logging
SubmissionScheduler.java
Removed unnecessary info logs from submission scheduler   

src/main/java/org/patinanetwork/codebloom/scheduled/submission/SubmissionScheduler.java

  • Removed an INFO log indicating users without a LeetCode username.
  • Removed an INFO log confirming a user's submission processing
    completion.
  • Slightly reformatted an existing comment for readability.
+2/-3     

@github-actions
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶

870 - Partially compliant

Compliant requirements:

  • Remove wasteful/cluttering logs from the codebase.

Non-compliant requirements:

Requires further human verification:

  • All tests have passed
  • PR successfully deployed to staging
  • Manual QA in dev and staging
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@github-actions
Copy link
Contributor

Title

870: get rid of cluttering logs


PR Type

Enhancement


Description

  • Removed verbose INFO logs.

  • Streamlined SubmissionScheduler output.

  • Enhanced log clarity and focus.


File Walkthrough

Relevant files
Logging
SubmissionScheduler.java
Streamline logging in `SubmissionScheduler`                           

src/main/java/org/patinanetwork/codebloom/scheduled/submission/SubmissionScheduler.java

  • Removed an INFO log message for users without a LeetCode username, as
    this is an expected condition.
  • Eliminated an INFO log message confirming completion of submission
    handling for each user to reduce verbosity.
  • Adjusted a multi-line comment for improved readability.
+2/-3     

@github-actions
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

**🎫 Ticket compliance analysis **

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@alfardil
Copy link
Collaborator Author

/deploy

@alfardil alfardil enabled auto-merge (rebase) March 19, 2026 18:20
@alfardil alfardil merged commit e8370b9 into main Mar 19, 2026
37 checks passed
@alfardil alfardil deleted the 870 branch March 19, 2026 18:32
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.

2 participants