rishabhdaim commented on code in PR #24:
URL:
https://github.com/apache/sling-org-apache-sling-event/pull/24#discussion_r1131875305
##########
src/main/java/org/apache/sling/event/impl/jobs/config/JobManagerConfiguration.java:
##########
@@ -92,6 +92,12 @@ public class JobManagerConfiguration {
description = "Specify the periodic interval in minutes (default
is 48h - use 0 to disable) after which " +
"removed jobs (ERROR or DROPPED) should be cleaned from
the repository.")
int cleanup_period() default 2880;
+
+ @AttributeDefinition(name = "Progress Log message's max count",
+ description = "Max number of log messages that can stored by
consumer to add information about current state of Job.\n" +
+ "Any attempt to add more information would result into
purging of the least recent messages." +
+ "default is -1 (to indicate infinite).")
Review Comment:
Yes, `0` would mean discarding all the log messages.
I didn't think we need to add this specifically to the documentation because
it's not a special case unlike `-1`.
But adding this to the documentation would make this more clear.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
