Update version #4.

https://sourceforge.net/u/winhvu/review/ci/ab6cf7db983cc97ad4e4b5e39327bd9c72004d11/

Major Changes:
1) Remove failed-recovery client is done in LOG application thread instead of 
recovery thread
2) Do not lock application thread while recovery thread is on-going if it is 
operating on successful recovery clients
3) Make `Search<xx>ByHandle()` to `FetchAndUpdateObjectState()` critical 
section.
4) Introduce ScopeData class, which is internally to lga_agent.cc, to avoid 
using `goto` - easy missing restore the data back if it runs out of scope (e.g: 
unlock recovery mutex, or restore the object state back).
5) Enhance documentation.

/Vu



---

** [tickets:#2412] log: refactor handling log client database in log agent**

**Status:** review
**Milestone:** 5.17.08
**Created:** Tue Apr 04, 2017 12:08 PM UTC by Canh Truong
**Last Updated:** Mon May 08, 2017 08:49 AM UTC
**Owner:** Vu Minh Nguyen


In log agent, there is a link list holding all log clients of an application 
process. Also, in each log client, there is an additional link list holding all 
log streams which belongs to each log client.

Adding, modifying or deleing the link lists' elements or on sub-items of the 
client dabases are distrubuted in a lot of places, this could easily cause 
troubles regarding race condition, deadlock, or risks when adding code that do 
changes the databases.

So, this ticket intends to remove that concern by doing:
1) Centralizing read/write accesses to the database to one place with its 
private mutex
2) Use C++ containters to contain and handle databases

And will push the ticket in 02 increments:
1) Convert agent code to C++ without touching any existing logic (looks like 
what AMF has done it in [#1673])
2) Do #1 and #2 above


---

Sent from sourceforge.net because [email protected] is 
subscribed to https://sourceforge.net/p/opensaf/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/opensaf/admin/tickets/options.  Or, if this is a 
mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-tickets mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets

Reply via email to