RFR: 8330077: Allow max number of events to be buffered to be configurable to avoid OVERFLOW_EVENT [v13]
Brian Burkhalter
bpb at openjdk.org
Wed Apr 24 15:39:31 UTC 2024
On Wed, 24 Apr 2024 08:18:23 GMT, Fabian Meumertzheim <duke at openjdk.org> wrote:
>> src/java.base/share/classes/java/nio/file/WatchService.java line 109:
>>
>>> 107: * scan a watched directory to get an updated list of the files in the
>>> 108: * directory. The limit for the pending events can be changed from its default
>>> 109: * with the system property {@code jdk.nio.file.WatchService.maxEventsPerPoll}
>>
>> @fmeum On this line 109, would you please change `@code` to `@systemProperty`?
>>
>>
>> --- a/src/java.base/share/classes/java/nio/file/WatchService.java
>> +++ b/src/java.base/share/classes/java/nio/file/WatchService.java
>> @@ -106,7 +106,8 @@
>> * special event is the trigger to re-examine the state of the object, e.g.
>> * scan a watched directory to get an updated list of the files in the
>> * directory. The limit for the pending events can be changed from its default
>> - * with the system property {@code jdk.nio.file.WatchService.maxEventsPerPoll}
>> + * with the system property
>> + * {@systemProperty jdk.nio.file.WatchService.maxEventsPerPoll}
>> * set to a value that parses as a positive integer. This may be useful in
>> * environments where there is a high volume of changes and where the impact
>> * of discarded events is high.
>> * ```
>
> Thanks, I didn't even know about this tag.
Neither did I, until @AlanBateman made me aware of it.
I will run some more tests and file the CSR.
Thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18739#discussion_r1578101555
More information about the nio-dev
mailing list