SWAT-engineering / SWAT-engineering/java-watch
Enqueue collections of events instead of individual events in `MacWatchKey`?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 1
- Forks
- 0
- Ø Merge
- 14 Std. 40 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
Events are enqueued to the list of pending events individually in MacWatchKey. It might be faster to, before enqueueing, collect all events that are available at that point (but not yet enqueued) and enqueue the collection. Note: This is an optimization of which the need/effectiveness currently is unclear.
Further background:
- Original comment
- In a previous version of the library, the poller used to submit a separate job to the thread pool for each incoming event. This didn't work well in situations with many events (e.g., in the torture tests). Currently, instead, the poller collects multiple events in each job. This feature of the poller might be strengthened by using "pre-collection" as suggested in this issue (or it might not; that's to be found out).
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Lies zunächst MacWatchKey und das bestehende Verhalten des poller beim Sammeln von Ereignissen. Führe die torture tests aus, um den im Issue beschriebenen Fall mit hoher Ereignisrate zu verstehen, und miss anschließend, ob das vorherige Sammeln verfügbarer Ereignisse ihn verbessert. Als abgeschlossen gilt die Arbeit, wenn die Auswirkung der Optimierung festgestellt und jede daraus resultierende Änderung anhand dieser Tests validiert wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- operating-systems, performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 28/100