[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