Hi all Carsten started to review the Sightly Engine API in SLING-4275 [1]. Since this issue covers 4 topics, I think it would be best to quickly discuss these topic in separate threads.
#1 Exceptions The root exception is SightlyException extends SlingException. As such it is a RuntimeException. Question is whether we need an exception hierarchy or not. The current exception types are thrown from different extension of the Sightly Engine such as RuntimeExtension instances or UseProviders. Most of the time these exceptions are caught and handled inside the Sightly Engine. As such it might be conceivable to actually collapse those exceptions into the root SightlyException. On the other hand: Are there exceptions defined that we expect to be caught by consumer code outside of the Sightly Engine ? Such exceptions probably make sense and should be kept. WDYT ? Regards Felix [1] https://issues.apache.org/jira/browse/SLING-4275
