[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #20540 [Metrics]: define log-levels for all java metrics-products
#20540: define log-levels for all java metrics-products
-------------------------+------------------------------
Reporter: iwakeh | Owner:
Type: enhancement | Status: needs_review
Priority: Medium | Milestone:
Component: Metrics | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------+------------------------------
Changes (by karsten):
* status: needs_information => needs_review
Comment:
There, I think I made [https://gitweb.torproject.org/user/karsten/metrics-
lib.git/commit/?h=task-20540&id=30113b528dcfca39b9ecc8179f195e9558dc428d
all changes as suggested].
And I changed that one log statement back to an
`IllegalArgumentException`, even though I think that's a stretch. (I
believe that we should only throw `RuntimeException`s at callers for
things they're clearly responsible for, like passing `null` to us where we
clearly said that they must not do such a thing. But the caller does not
have exclusive control over the file system, so it might not have been
them who created the non-directory file. Yeah, a stretch, gray area, edge
case... I don't know the final answer yet.)
Want to take another look?
Should I continue with the other metrics-lib classes, like
`DescriptorReaderImpl`?
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20540#comment:11>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs