Fine with me, provided we have tested the new impl in and out. HA-JNDI is used a lot
The new implementation has been running as the default implementation in head for a few months. If people are running cluster environments using the head code base and using HA-JNDI capabilities, then it should be well-exercised, otherwise not.
It satisfies the HA-JNDI unit tests that I wrote before changing the original implementation; the tests are fairly simple though.
Performance testing shows that there's no appreciable performance difference between the two implementations.
The old implementation has now been removed from JBoss 5.0.0alpha. I've revised the HA-JNDI wiki page so that it no longer describes how to use the former implementation.