[tickets] [opensaf:tickets] #2554 dtm: Re-factor the inter-node receive logic

[Anders Widell via Opensaf-tickets]

- **status**: review --> fixed
- **Comment**:

commit 2b4b4b2fd0f70381832093dd09d4a75208c0 (HEAD -> develop, 
origin/develop, ticket-2554)
Author: Anders Widell 
Date:   Fri Aug 18 12:48:17 2017 +0200

dtm: Re-factor the inter-node receive logic [#2554]

The DTM receive logic was unnecessarily complex: it called recv() in many
different places and had conditional statements that executed different 
pieces
of code depending on how many bytes that had been received on the stream 
socket
so far. The code has now been refactored so that it uses only one single 
recv()
statement and one single check to determine if it has received a full 
message or
not.




---

** [tickets:#2554] dtm: Re-factor the inter-node receive logic**

**Status:** fixed
**Milestone:** 5.17.10
**Created:** Wed Aug 16, 2017 09:00 AM UTC by Anders Widell
**Last Updated:** Mon Aug 21, 2017 08:55 AM UTC
**Owner:** Anders Widell


The DTM receive logic is unnecessarily complex: it calls recv() in many 
different places and has conditional statements that executes different pieces 
of code depending on how many bytes have been received on the stream socket so 
far. It should be refactored so that it uses only one single recv() statement 
and one single check to determine if we have received a full message or not.


---

Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net 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
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets

[tickets] [opensaf:tickets] #2554 dtm: Re-factor the inter-node receive logic

[Anders Widell via Opensaf-tickets]

- **status**: accepted --> review



---

** [tickets:#2554] dtm: Re-factor the inter-node receive logic**

**Status:** review
**Milestone:** 5.17.10
**Created:** Wed Aug 16, 2017 09:00 AM UTC by Anders Widell
**Last Updated:** Thu Aug 17, 2017 01:37 PM UTC
**Owner:** Anders Widell


The DTM receive logic is unnecessarily complex: it calls recv() in many 
different places and has conditional statements that executes different pieces 
of code depending on how many bytes have been received on the stream socket so 
far. It should be refactored so that it uses only one single recv() statement 
and one single check to determine if we have received a full message or not.


---

Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net 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
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets

[tickets] [opensaf:tickets] #2554 dtm: Re-factor the inter-node receive logic

[Anders Widell via Opensaf-tickets]

- **status**: assigned --> accepted



---

** [tickets:#2554] dtm: Re-factor the inter-node receive logic**

**Status:** accepted
**Milestone:** 5.17.10
**Created:** Wed Aug 16, 2017 09:00 AM UTC by Anders Widell
**Last Updated:** Wed Aug 16, 2017 09:00 AM UTC
**Owner:** Anders Widell


The DTM receive logic is unnecessarily complex: it calls recv() in many 
different places and has conditional statements that executes different pieces 
of code depending on how many bytes have been received on the stream socket so 
far. It should be refactored so that it uses only one single recv() statement 
and one single check to determine if we have received a full message or not.


---

Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net 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
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets

[tickets] [opensaf:tickets] #2554 dtm: Re-factor the inter-node receive logic

[Anders Widell via Opensaf-tickets]

---

** [tickets:#2554] dtm: Re-factor the inter-node receive logic**

**Status:** assigned
**Milestone:** 5.17.10
**Created:** Wed Aug 16, 2017 09:00 AM UTC by Anders Widell
**Last Updated:** Wed Aug 16, 2017 09:00 AM UTC
**Owner:** Anders Widell


The DTM receive logic is unnecessarily complex: it calls recv() in many 
different places and has conditional statements that executes different pieces 
of code depending on how many bytes have been received on the stream socket so 
far. It should be refactored so that it uses only one single recv() statement 
and one single check to determine if we have received a full message or not.


---

Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net 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
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets