search

claeis / ilivalidator

INTERLIS validator
14 stars 21 forks source link

Measure & log complete validation time #344

Closed Philippluca closed 2 years ago

Philippluca commented 2 years ago

Ref: https://github.com/claeis/ilivalidator/issues/317 Requires iox-ili Stopwatch added in https://github.com/claeis/iox-ili/pull/110

claeis commented 2 years ago

Ist m.E. überflüssig mit dem bereits implementierten neuen --logtime

Philippluca commented 2 years ago

Wenn ich das richtig verstanden habe, dann wird mit --logtime der Log Eintrag im Logfile um einen Timestamp erweitert. Für den Anwender des UI oder der Konsole ohne Logfile glaube ich, dass die ausgerechnete Dauer direkt im Konsolen / UI output einen Mehrwert bietet. Kannst du diesen Mehrwert so auch nachvollziehen?

claeis commented 2 years ago

Ich sehe für Nutzer keinen Mehrwert. Aber wenn das hier (mittels StopWatch) gemessen und ausgegeben wird, stört es mich nicht.

claeis commented 2 years ago

Es führt höchstens zu Fragen, warum es manchmal schneller und manchmal länger dauert.

olivergrimm commented 2 years ago

ich fände es gut wenn dies künftig ausgegeben wird. Es fördert bei den Anwender:innen das Bewusstsein dafür, was alles validiert wird und wie kostenintensiv ist.