[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #8607 [Stem]: Controller's cache isn't thread safe



#8607: Controller's cache isn't thread safe
-----------------------------+----------------------------------------------
 Reporter:  atagar           |          Owner:  atagar      
     Type:  defect           |         Status:  needs_review
 Priority:  normal           |      Milestone:              
Component:  Stem             |        Version:              
 Keywords:  controller easy  |         Parent:              
   Points:                   |   Actualpoints:              
-----------------------------+----------------------------------------------
Changes (by axitkhurana):

  * status:  needs_revision => needs_review


Comment:

 Thanks for taking a quick look, the CONF_CHANGED fix is failing one of the
 integration tests, I'll fix it, add a test and post it on it's ticket:
 https://trac.torproject.org/7713 . Sorry about the extra indentation, my
 vimrc was configured for 4 spaces per indent then.

 For the concurrency fix, my earlier commit was breaking integration tests,
 but now they are passing. Also, capitalized the first letter of commit
 messages and added ticket link. Please take a look here:
 https://github.com/axitkhurana/stem/commit/4e0ec8cf702f5db6449d49cccc262fb8b6b5f97f

 Thanks :)
 Akshit

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/8607#comment:3>
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