Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

Describe your changes:

Applied code formatting fixes to address checkstyle violations in ServicesStatusJobHandler.java. The spotless formatter corrected indentation in a try-catch block that was improperly aligned.

Changes:

  • Fixed indentation in ServicesStatusJobHandler.create() method's try-catch block

Type of change:

  • Improvement

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Contributor Author

Copilot AI commented Jan 15, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • repository.apache.org
    • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.12/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.12/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.12 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.12/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/OpenMetadata/OpenMetadata org.codehaus.plexus.classworlds.launcher.Launcher spotless:apply (dns block)
  • s3.amazonaws.com
    • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.12/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.12/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.12 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.12/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/OpenMetadata/OpenMetadata org.codehaus.plexus.classworlds.launcher.Launcher spotless:apply (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix clean Pipeline Service Client health job in favor of on-demand checks Fix code formatting with mvn spotless:apply Jan 15, 2026
Copilot AI requested a review from pmbrull January 15, 2026 10:05
@pmbrull pmbrull marked this pull request as ready for review January 15, 2026 10:12
@pmbrull pmbrull requested a review from a team as a code owner January 15, 2026 10:12
@pmbrull pmbrull merged commit 62492e4 into issue-2493 Jan 15, 2026
2 of 18 checks passed
@pmbrull pmbrull deleted the copilot/sub-pr-25228 branch January 15, 2026 10:12
pmbrull added a commit that referenced this pull request Jan 16, 2026
#25228)

* FIX #2493 - Clean Pipeline Service Client health job in favor of on-demand checks

* Remove comment on scheduled health check

Removed the comment about the scheduled pipeline service health check.

* FIX #2493 - Clean Pipeline Service Client health job in favor of on-demand checks

* FIX #2493 - Clean Pipeline Service Client health job in favor of on-demand checks

* Addressing feedback

Co-authored-by: pmbrull <[email protected]>

* Fix code formatting with mvn spotless:apply (#25316)

* Initial plan

* Fix code formatting with mvn spotless:apply

Co-authored-by: pmbrull <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: pmbrull <[email protected]>

---------

Co-authored-by: Gitar <[email protected]>
Co-authored-by: pmbrull <[email protected]>
Co-authored-by: Copilot <[email protected]>
github-actions bot pushed a commit that referenced this pull request Jan 16, 2026
#25228)

* FIX #2493 - Clean Pipeline Service Client health job in favor of on-demand checks

* Remove comment on scheduled health check

Removed the comment about the scheduled pipeline service health check.

* FIX #2493 - Clean Pipeline Service Client health job in favor of on-demand checks

* FIX #2493 - Clean Pipeline Service Client health job in favor of on-demand checks

* Addressing feedback

Co-authored-by: pmbrull <[email protected]>

* Fix code formatting with mvn spotless:apply (#25316)

* Initial plan

* Fix code formatting with mvn spotless:apply

Co-authored-by: pmbrull <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: pmbrull <[email protected]>

---------

Co-authored-by: Gitar <[email protected]>
Co-authored-by: pmbrull <[email protected]>
Co-authored-by: Copilot <[email protected]>
(cherry picked from commit 922389a)
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