Skip to content

Keep the recompiling message at the INFO level when user explicitly requested that information - #1108

Open
desruisseaux wants to merge 1 commit into
apache:masterfrom
Geomatys:info-level-when-requested
Open

Keep the recompiling message at the INFO level when user explicitly requested that information#1108
desruisseaux wants to merge 1 commit into
apache:masterfrom
Geomatys:info-level-when-requested

Conversation

@desruisseaux

Copy link
Copy Markdown
Contributor

Keep the recompiling message at the INFO level when user explicitly requested that information. This pull request narrows the scope of #1089 to the case when the user did not requested that information.

@elharo elharo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A trace level below debug level sounds useful. I think we have defaulted to showing far too much logging that is never looked at and merely obscures real issues.

My rule of thumb is that every log message should be interesting, actionable, and useful, especially at info level or higher. It is not enough that someone might sometime want to see the log. To log at info level we need to be confident that most people will want to see it most of the time.

More needles, less hay. :-)

* Whether to provide more details about why a module is rebuilt.
*/
private final boolean showCompilationChanges;
final boolean showCompilationChanges;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How does the user set this? Is there a paremter somewhere?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, this is a parameter in CompilerMojo. This parameter already existed in the Maven 3 plugin.

@desruisseaux desruisseaux added the java Pull requests that update Java code label Aug 5, 2026
@desruisseaux desruisseaux added this to the 4.0.0-beta-5 milestone Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants