]
Bela Ban commented on JGRP-2205:
--------------------------------
Actually, both setting the local address and copying the message is done in the transport,
so I'm going to remove {{loopback()}} altogether and simply call
{{down_prot.down(msg)}}.
DISCARD ignores the DONT_LOOPBACK transient flag
------------------------------------------------
Key: JGRP-2205
URL:
https://issues.jboss.org/browse/JGRP-2205
Project: JGroups
Issue Type: Bug
Affects Versions: 4.0.4
Reporter: Dan Berindei
Assignee: Bela Ban
Fix For: 4.0.5
When {{discard_all = true}}, {{DISCARD}} does its own loopback, and doesn't check for
{{DONT_LOOPBACK}} like {{TP}}. It always sends the message back up, even if
{{excludeItself = false}}.
If possible, {{DISCARD}} should just set the message destination to the local address and
pass the message down. That way, {{TP}} would decide make the loopback decision, and using
the {{TP}} thread pool would also make the thread name nicer in the logs. (Currently the
thread name is {{Thread-n}}, which means searching for the test name in our test
suite's log misses some messages.)