You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/08/19 08:09:22 UTC
[GitHub] [incubator-druid] leventov opened a new issue #8333: Statically
prohibit missing format parameters in Druid's exception classes
leventov opened a new issue #8333: Statically prohibit missing format parameters in Druid's exception classes
URL: https://github.com/apache/incubator-druid/issues/8333
https://youtrack.jetbrains.com/issue/IDEA-175327 doesn't allow to prevent bugs like fixed in #8331 using IntelliJ's "Malformed format string" inspection. However, there is an easy workaround: make constructors in all Druid's exception classes like `RE`, `ISE`, `IAE`, etc. private, expose static factory methods (e. g. `RE.of()`), and put them into the MalformedFormatString inspection config.
This would be a follow-up of #4474.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org