[
https://issues.apache.org/jira/browse/SLING-10283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Angela Schreiber updated SLING-10283:
-------------------------------------
Description:
Extracted from review finding for SLING-10281:
IMHO it would be better to throw dedicated repo-init-runtime-exceptions that
come with an error-code and a message instead of throwing RuntimeException,
IllegalStateException and IllegalArgumentException.
that would allow to document the various error codes along with explanations of
the root cause and instructions on how to fix the corresponding repo-init
statements.
[~kwin], fyi.
was:
Extracted from review finding for SLING-10281:
IMHO it would be better to throw dedicated repo-init-runtime-exceptions that
come with an error-code and a message instead of throwing RuntimeException,
IllegalStateException and IllegalArgumentException.
that would allow to document the various error codes along with explanations of
the root cause and instructions on how to fix the corresponding repo-init
statements.
[~kwindszus], fyi.
> Defined dedicated RepoInit RuntimeException instead of throwing generic
> RuntimeException/IllegalStateException/IllegalArgumentException
> ---------------------------------------------------------------------------------------------------------------------------------------
>
> Key: SLING-10283
> URL: https://issues.apache.org/jira/browse/SLING-10283
> Project: Sling
> Issue Type: Improvement
> Components: Repoinit
> Reporter: Angela Schreiber
> Priority: Major
>
> Extracted from review finding for SLING-10281:
> IMHO it would be better to throw dedicated repo-init-runtime-exceptions that
> come with an error-code and a message instead of throwing RuntimeException,
> IllegalStateException and IllegalArgumentException.
> that would allow to document the various error codes along with explanations
> of the root cause and instructions on how to fix the corresponding repo-init
> statements.
> [~kwin], fyi.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
