Bug 1015695 - MapMessage does not accept null values
Summary: MapMessage does not accept null values
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: HornetQ
Version: 6.1.1
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ER6
: EAP 6.2.0
Assignee: Clebert Suconic
QA Contact: Martin Svehla
Russell Dickenson
URL:
Whiteboard:
Depends On: 1016141
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-04 20:59 UTC by Fernando Nasser
Modified: 2013-12-15 16:13 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-15 16:13:08 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Fernando Nasser 2013-10-04 20:59:28 UTC
Description of problem:

MapMessage m = session.createMapMessage();
m.setObject("value", null);

results in a 

javax.jms.MessageFormatException: Invalid object type.


Version-Release number of selected component (if applicable):

2.3.1

How reproducible:

Always

Steps to Reproduce:
1. Try and set the value of a property in a MapMessage to null 
2.
3.

Actual results:

MessageFormatException


Expected results:

property value set to null
Additional info:

Comment 1 Fernando Nasser 2013-10-04 21:12:02 UTC
Assigned to Justin at Clebert's request

Comment 2 Clebert Suconic 2013-10-05 15:20:15 UTC
I already did it actually.. boredom on friday night

Comment 3 Clebert Suconic 2013-10-05 15:20:45 UTC
https://github.com/hornetq/hornetq/pull/1319

Comment 4 Fernando Nasser 2013-10-06 15:40:41 UTC
I am setting the devel_ack on behalf of Clebert (who has already backported the fix)

Comment 5 Fernando Nasser 2013-10-07 13:50:10 UTC
Hi Miroslav or Rosta,,

We have two options, 

1) cherry pick commit 8b6971d64d8153443bbbd15e063c1a3154ec0bbc from HQ 2.3.x branch and patch the 2.3.8-redhat-2 we have and create a 2.3.8-redhat-3

2) Get a 2.3.9 tag from Clebert.  He says:

[09:44]  <clebert> fnasser:  the fixes I have so far on 2.3.x are small risk

Can we go (2) or do you prefer we try (1)?

Comment 6 Clebert Suconic 2013-10-07 14:06:22 UTC
If we do a release of 2.3 now we would include these fixes:

https://bugzilla.redhat.com/show_bug.cgi?id=959641 - partial failure on RA
https://bugzilla.redhat.com/show_bug.cgi?id=1009409 - exceptions on shutdown with bridge
https://bugzilla.redhat.com/show_bug.cgi?id=1009530 - interrupt of shutdown
https://community.jboss.org/thread/232591 - fixing typo accordingly to user
HORNETQ-431 - Large Messages Files NOT DELETED on unbounded address (BZ TBD)

https://bugzilla.redhat.com/show_bug.cgi?id=1013884 Consumer.close() shouldn't fire redelivery check


https://bugzilla.redhat.com/show_bug.cgi?id=1013777 - Fixing intermittent failure on BackupSyncLargeMessageTest::testDeleteLargeMessages


All of them are minor fixes. Fixing tests or things that are unlikely to cause any disruption on the tests. 2.3.x is a stable branch for us after all.

Comment 7 Fernando Nasser 2013-10-07 14:16:42 UTC
We are probably also interested in #1009409 and #1013884 as well.

Comment 8 Miroslav Novak 2013-10-07 14:34:18 UTC
I'm ok with adding those fixes. 

Cheers,
Mirek

Comment 11 Martin Svehla 2013-10-17 09:13:31 UTC
Tested with EAP 6.2.0.ER6 / HornetQ 2.3.9.Final


Note You need to log in before you can comment on or make changes to this bug.