You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "andrewmkhoury (via GitHub)" <gi...@apache.org> on 2023/04/21 15:29:58 UTC

[GitHub] [sling-org-apache-sling-event] andrewmkhoury commented on a diff in pull request #26: SLING-11831 - Allow setting job properties for custom job state

andrewmkhoury commented on code in PR #26:
URL: https://github.com/apache/sling-org-apache-sling-event/pull/26#discussion_r1173911475


##########
src/main/java/org/apache/sling/event/impl/jobs/tasks/CleanUpTask.java:
##########
@@ -181,6 +181,11 @@ public void log(String message, Object... args) {
 
                         }
 
+                        @Override
+                        public void setProperty(String name, Object value) {
+

Review Comment:
   All the methods of that JobExecutionContext had no comments.  So I addedd a comment at the top of the class explaining why.



-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org