DominikSuess commented on a change in pull request #3:
URL: 
https://github.com/apache/sling-org-apache-sling-jcr-packageinit/pull/3#discussion_r565189069



##########
File path: 
src/main/java/org/apache/sling/jcr/packageinit/impl/ExecutionPlanRepoInitializer.java
##########
@@ -163,6 +164,14 @@ public void processRepository(SlingRepository 
slingRepository) throws Exception
                         builder.with(session);
                         ExecutionPlan xplan = builder.execute();
                         if (xplan.getTasks().size() > 0) {
+                            if (xplan.hasErrors()) {
+                                for (PackageTask task : xplan.getTasks()) {
+                                    if 
(PackageTask.State.ERROR.equals(task.getState())){
+                                        logger.error("Error during 
installation of {}: {}", task.getPackageId().toString(), 
task.getError().toString());

Review comment:
       @kwin  ah yes makes sense.. just pushed a corresponding update




----------------------------------------------------------------
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:
[email protected]


Reply via email to