[jira] [Commented] (ARTEMIS-1498) [interop] Openwire internal headers should not be part of message properties

2018-03-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413388#comment-16413388
 ] 

ASF GitHub Bot commented on ARTEMIS-1498:
-

Github user michaelandrepearce commented on the issue:

https://github.com/apache/activemq-artemis/pull/1793
  
@RaiSaurabh could you rebase your branch on current master then, for me? 


> [interop] Openwire internal headers should not be part of message properties
> 
>
> Key: ARTEMIS-1498
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1498
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>  Components: OpenWire
> Environment: interoperability between Openwire -> AMQP/Core
>Reporter: Michal Toth
>Priority: Minor
>
> Sending an empty message from Openwire JMS client to AMQP or Core receiver 
> client shows extra "internal" headers (__HDR_*) in message properties. 
> They should not be there, as these are internal for broker in my opinion.
> Openwire sender
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aoc7.jar sender  --timeout 5 
> --log-msgs interop --broker tcp://localhost:61616 --address queue_name_\$ 
> --count 1  --conn-username admin --conn-password admin
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': 
> 'dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1:1:1:1', 
> 'user-id':None, 'address': 'queue_name_$', 'subject': None, 'reply-to': None, 
> 'correlation-id': None, 'content-type': None, 'content-encoding': None, 
> 'absolute-expiry-time': 0, 'creation-time': 1509702974798, 'group-id': None, 
> 'group-sequence': 0, 'reply-to-group-id': None, 'properties': {}, 'content': 
> None, 'type': None}
> {noformat}
> AMQP/Core receiver
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aac1.jar receiver  --timeout 5 
> --log-msgs interop --broker localhost:5672 --address queue_name_\$ --count 1  
> --conn-username admin --conn-password admin
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': None, 'user-id': None, 'address': 'queue_name_$', 
> 'subject': None, 'reply-to': None, 'correlation-id': None, 'content-type': 
> None, 'content-encoding': None, 'absolute-expiry-time': 0, 'creation-time': 
> 1509702974798, 'group-id': None, 'group-sequence': 0, 'reply-to-group-id': 
> None, 'properties': {'__HDR_COMMAND_ID': 5, 'JMSXDeliveryCount': 1, 
> '__HDR_ARRIVAL': 0, '__HDR_MESSAGE_ID': 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01,
>  '__HDR_GROUP_SEQUENCE': 0, '__HDR_PRODUCER_ID': 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01,
>  '__HDR_DROPPABLE': False, '__HDR_BROKER_IN_TIME': 1509702974800}, 'content': 
> None}
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (ARTEMIS-1498) [interop] Openwire internal headers should not be part of message properties

2018-03-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413389#comment-16413389
 ] 

ASF GitHub Bot commented on ARTEMIS-1498:
-

Github user RaiSaurabh commented on the issue:

https://github.com/apache/activemq-artemis/pull/1793
  
Ok @michaelandrepearce .


> [interop] Openwire internal headers should not be part of message properties
> 
>
> Key: ARTEMIS-1498
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1498
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>  Components: OpenWire
> Environment: interoperability between Openwire -> AMQP/Core
>Reporter: Michal Toth
>Priority: Minor
>
> Sending an empty message from Openwire JMS client to AMQP or Core receiver 
> client shows extra "internal" headers (__HDR_*) in message properties. 
> They should not be there, as these are internal for broker in my opinion.
> Openwire sender
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aoc7.jar sender  --timeout 5 
> --log-msgs interop --broker tcp://localhost:61616 --address queue_name_\$ 
> --count 1  --conn-username admin --conn-password admin
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': 
> 'dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1:1:1:1', 
> 'user-id':None, 'address': 'queue_name_$', 'subject': None, 'reply-to': None, 
> 'correlation-id': None, 'content-type': None, 'content-encoding': None, 
> 'absolute-expiry-time': 0, 'creation-time': 1509702974798, 'group-id': None, 
> 'group-sequence': 0, 'reply-to-group-id': None, 'properties': {}, 'content': 
> None, 'type': None}
> {noformat}
> AMQP/Core receiver
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aac1.jar receiver  --timeout 5 
> --log-msgs interop --broker localhost:5672 --address queue_name_\$ --count 1  
> --conn-username admin --conn-password admin
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': None, 'user-id': None, 'address': 'queue_name_$', 
> 'subject': None, 'reply-to': None, 'correlation-id': None, 'content-type': 
> None, 'content-encoding': None, 'absolute-expiry-time': 0, 'creation-time': 
> 1509702974798, 'group-id': None, 'group-sequence': 0, 'reply-to-group-id': 
> None, 'properties': {'__HDR_COMMAND_ID': 5, 'JMSXDeliveryCount': 1, 
> '__HDR_ARRIVAL': 0, '__HDR_MESSAGE_ID': 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01,
>  '__HDR_GROUP_SEQUENCE': 0, '__HDR_PRODUCER_ID': 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01,
>  '__HDR_DROPPABLE': False, '__HDR_BROKER_IN_TIME': 1509702974800}, 'content': 
> None}
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (ARTEMIS-1498) [interop] Openwire internal headers should not be part of message properties

2018-03-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413386#comment-16413386
 ] 

ASF GitHub Bot commented on ARTEMIS-1498:
-

Github user RaiSaurabh commented on the issue:

https://github.com/apache/activemq-artemis/pull/1793
  
@michaelandrepearce Yes, I did.


> [interop] Openwire internal headers should not be part of message properties
> 
>
> Key: ARTEMIS-1498
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1498
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>  Components: OpenWire
> Environment: interoperability between Openwire -> AMQP/Core
>Reporter: Michal Toth
>Priority: Minor
>
> Sending an empty message from Openwire JMS client to AMQP or Core receiver 
> client shows extra "internal" headers (__HDR_*) in message properties. 
> They should not be there, as these are internal for broker in my opinion.
> Openwire sender
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aoc7.jar sender  --timeout 5 
> --log-msgs interop --broker tcp://localhost:61616 --address queue_name_\$ 
> --count 1  --conn-username admin --conn-password admin
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': 
> 'dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1:1:1:1', 
> 'user-id':None, 'address': 'queue_name_$', 'subject': None, 'reply-to': None, 
> 'correlation-id': None, 'content-type': None, 'content-encoding': None, 
> 'absolute-expiry-time': 0, 'creation-time': 1509702974798, 'group-id': None, 
> 'group-sequence': 0, 'reply-to-group-id': None, 'properties': {}, 'content': 
> None, 'type': None}
> {noformat}
> AMQP/Core receiver
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aac1.jar receiver  --timeout 5 
> --log-msgs interop --broker localhost:5672 --address queue_name_\$ --count 1  
> --conn-username admin --conn-password admin
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': None, 'user-id': None, 'address': 'queue_name_$', 
> 'subject': None, 'reply-to': None, 'correlation-id': None, 'content-type': 
> None, 'content-encoding': None, 'absolute-expiry-time': 0, 'creation-time': 
> 1509702974798, 'group-id': None, 'group-sequence': 0, 'reply-to-group-id': 
> None, 'properties': {'__HDR_COMMAND_ID': 5, 'JMSXDeliveryCount': 1, 
> '__HDR_ARRIVAL': 0, '__HDR_MESSAGE_ID': 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01,
>  '__HDR_GROUP_SEQUENCE': 0, '__HDR_PRODUCER_ID': 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01,
>  '__HDR_DROPPABLE': False, '__HDR_BROKER_IN_TIME': 1509702974800}, 'content': 
> None}
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (ARTEMIS-1498) [interop] Openwire internal headers should not be part of message properties

2018-03-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413384#comment-16413384
 ] 

ASF GitHub Bot commented on ARTEMIS-1498:
-

Github user michaelandrepearce commented on the issue:

https://github.com/apache/activemq-artemis/pull/1793
  
@RaiSaurabh im struggling to rebase this on top of master. Did you rebase 
after @franz1981's work?


> [interop] Openwire internal headers should not be part of message properties
> 
>
> Key: ARTEMIS-1498
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1498
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>  Components: OpenWire
> Environment: interoperability between Openwire -> AMQP/Core
>Reporter: Michal Toth
>Priority: Minor
>
> Sending an empty message from Openwire JMS client to AMQP or Core receiver 
> client shows extra "internal" headers (__HDR_*) in message properties. 
> They should not be there, as these are internal for broker in my opinion.
> Openwire sender
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aoc7.jar sender  --timeout 5 
> --log-msgs interop --broker tcp://localhost:61616 --address queue_name_\$ 
> --count 1  --conn-username admin --conn-password admin
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': 
> 'dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1:1:1:1', 
> 'user-id':None, 'address': 'queue_name_$', 'subject': None, 'reply-to': None, 
> 'correlation-id': None, 'content-type': None, 'content-encoding': None, 
> 'absolute-expiry-time': 0, 'creation-time': 1509702974798, 'group-id': None, 
> 'group-sequence': 0, 'reply-to-group-id': None, 'properties': {}, 'content': 
> None, 'type': None}
> {noformat}
> AMQP/Core receiver
> {noformat}
> $ java  -jar /var/dtests/node_data/clients/aac1.jar receiver  --timeout 5 
> --log-msgs interop --broker localhost:5672 --address queue_name_\$ --count 1  
> --conn-username admin --conn-password admin
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01
> Unsupported object type org.apache.qpid.proton.amqp.Binary 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01
> {'durable': True, 'priority': 4, 'ttl': 0, 'first-acquirer': False, 
> 'delivery-count': 0, 'id': None, 'user-id': None, 'address': 'queue_name_$', 
> 'subject': None, 'reply-to': None, 'correlation-id': None, 'content-type': 
> None, 'content-encoding': None, 'absolute-expiry-time': 0, 'creation-time': 
> 1509702974798, 'group-id': None, 'group-sequence': 0, 'reply-to-group-id': 
> None, 'properties': {'__HDR_COMMAND_ID': 5, 'JMSXDeliveryCount': 1, 
> '__HDR_ARRIVAL': 0, '__HDR_MESSAGE_ID': 
> \x00\x00\x00Jn\x02\xae\x02{\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01\x00\x01,
>  '__HDR_GROUP_SEQUENCE': 0, '__HDR_PRODUCER_ID': 
> \x00\x00\x00F{\x01+\x00=ID:dhcp-145-96.lab.eng.brq.redhat.com-33646-1509702974536-1:1\x00\x01\x00\x01,
>  '__HDR_DROPPABLE': False, '__HDR_BROKER_IN_TIME': 1509702974800}, 'content': 
> None}
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (AMQ-6938) KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing journal files

2018-03-25 Thread Tim Bain (JIRA)

 [ 
https://issues.apache.org/jira/browse/AMQ-6938?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tim Bain updated AMQ-6938:
--
Description: 
It was reported on the user mailing list 
([http://activemq.2283324.n4.nabble.com/Re-failed-to-start-ActiveMQ-td4737631.html)]
 that a user is seeing the broker fail to start due to "java.io.IOException: 
Could not locate data file /data/kahadb/db-531.log" even though 
ignoreMissingJournalFiles is set to true. Presumably the purpose of that option 
is to allow the broker to ignore missing journal files (otherwise we chose 
poorly when naming the option and then failed to document it adequately on the 
wiki), which means the option doesn't work properly in all situations.

Looking at the code on master as of 3/25/2018, 
org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
journal.getNextLocation(null) on line 677, which will throw the exception in 
question when DataFile.getDataFile() is called. This call to 
journal.getNextLocation() occurs outside of the while loop, whereas the 
ignoreMissingJournalFiles value is only checked in the catch block within the 
while loop, so we do not consider the ignoreMissingJournalFiles value in the 
call to getNextLocation(), even though missing journal files can cause 
getNextLocation() to fail.

Note that the file is missing because the user manually deleted it in an effort 
to work around AMQ-6931, but that should not be relevant to the behavior that's 
being seen.

  was:
It was reported on the user mailing list 
([http://activemq.2283324.n4.nabble.com/Re-failed-to-start-ActiveMQ-td4737631.html)]
 that a user is seeing the broker fail to start due to "java.io.IOException: 
Could not locate data file /data/kahadb/db-531.log" even though 
ignoreMissingJournalFiles is set to true. Presumably the purpose of that option 
is to allow the broker to ignore missing journal files (otherwise we chose 
poorly when naming the option and then failed to document it adequately on the 
wiki), which means the option doesn't work properly in all situations.

Looking at the code on master as of 3/25/2018, 
org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
journal.getNextLocation(null) on line 677, which will throw the exception in 
question when DataFile.getDataFile() is called. This call to 
journal.getNextLocation() occurs outside of the while loop, whereas the 
ignoreMissingJournalFiles value is only checked in the catch block within the 
while loop, so we do not consider the ignoreMissingJournalFiles value in the 
call to getNextLocation(), even though missing journal files can cause 
getNextLocation() to fail.


> KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing 
> journal files
> -
>
> Key: AMQ-6938
> URL: https://issues.apache.org/jira/browse/AMQ-6938
> Project: ActiveMQ
>  Issue Type: Bug
>  Components: KahaDB
>Affects Versions: 5.13.1
>Reporter: Tim Bain
>Priority: Major
>
> It was reported on the user mailing list 
> ([http://activemq.2283324.n4.nabble.com/Re-failed-to-start-ActiveMQ-td4737631.html)]
>  that a user is seeing the broker fail to start due to "java.io.IOException: 
> Could not locate data file /data/kahadb/db-531.log" even though 
> ignoreMissingJournalFiles is set to true. Presumably the purpose of that 
> option is to allow the broker to ignore missing journal files (otherwise we 
> chose poorly when naming the option and then failed to document it adequately 
> on the wiki), which means the option doesn't work properly in all situations.
> Looking at the code on master as of 3/25/2018, 
> org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
> journal.getNextLocation(null) on line 677, which will throw the exception in 
> question when DataFile.getDataFile() is called. This call to 
> journal.getNextLocation() occurs outside of the while loop, whereas the 
> ignoreMissingJournalFiles value is only checked in the catch block within the 
> while loop, so we do not consider the ignoreMissingJournalFiles value in the 
> call to getNextLocation(), even though missing journal files can cause 
> getNextLocation() to fail.
> Note that the file is missing because the user manually deleted it in an 
> effort to work around AMQ-6931, but that should not be relevant to the 
> behavior that's being seen.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (AMQ-6938) KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing journal files

2018-03-25 Thread Tim Bain (JIRA)

 [ 
https://issues.apache.org/jira/browse/AMQ-6938?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tim Bain updated AMQ-6938:
--
Description: 
It was reported on the user mailing list 
([http://activemq.2283324.n4.nabble.com/Re-failed-to-start-ActiveMQ-td4737631.html)]
 that a user is seeing the broker fail to start due to "java.io.IOException: 
Could not locate data file /data/kahadb/db-531.log" even though 
ignoreMissingJournalFiles is set to true. Presumably the purpose of that option 
is to allow the broker to ignore missing journal files (otherwise we chose 
poorly when naming the option and then failed to document it adequately on the 
wiki), which means the option doesn't work properly in all situations.

Looking at the code on master as of 3/25/2018, 
org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
journal.getNextLocation(null) on line 677, which will throw the exception in 
question when DataFile.getDataFile() is called. This call to 
journal.getNextLocation() occurs outside of the while loop, whereas the 
ignoreMissingJournalFiles value is only checked in the catch block within the 
while loop, so we do not consider the ignoreMissingJournalFiles value in the 
call to getNextLocation(), even though missing journal files can cause 
getNextLocation() to fail.

  was:
It was reported on the user mailing list that a user is seeing the broker fail 
to start due to "java.io.IOException: Could not locate data file 
/data/kahadb/db-531.log" even though ignoreMissingJournalFiles is set to true. 
Presumably the purpose of that option is to allow the broker to ignore missing 
journal files (otherwise we chose poorly when naming the option and then failed 
to document it adequately on the wiki), which means the option doesn't work 
properly in all situations.

Looking at the code on master as of 3/25/2018, 
org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
journal.getNextLocation(null) on line 677, which will throw the exception in 
question when DataFile.getDataFile() is called. This call to 
journal.getNextLocation() occurs outside of the while loop, whereas the 
ignoreMissingJournalFiles value is only checked in the catch block within the 
while loop, so we do not consider the ignoreMissingJournalFiles value in the 
call to getNextLocation(), even though missing journal files can cause 
getNextLocation() to fail.


> KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing 
> journal files
> -
>
> Key: AMQ-6938
> URL: https://issues.apache.org/jira/browse/AMQ-6938
> Project: ActiveMQ
>  Issue Type: Bug
>  Components: KahaDB
>Affects Versions: 5.13.1
>Reporter: Tim Bain
>Priority: Major
>
> It was reported on the user mailing list 
> ([http://activemq.2283324.n4.nabble.com/Re-failed-to-start-ActiveMQ-td4737631.html)]
>  that a user is seeing the broker fail to start due to "java.io.IOException: 
> Could not locate data file /data/kahadb/db-531.log" even though 
> ignoreMissingJournalFiles is set to true. Presumably the purpose of that 
> option is to allow the broker to ignore missing journal files (otherwise we 
> chose poorly when naming the option and then failed to document it adequately 
> on the wiki), which means the option doesn't work properly in all situations.
> Looking at the code on master as of 3/25/2018, 
> org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
> journal.getNextLocation(null) on line 677, which will throw the exception in 
> question when DataFile.getDataFile() is called. This call to 
> journal.getNextLocation() occurs outside of the while loop, whereas the 
> ignoreMissingJournalFiles value is only checked in the catch block within the 
> while loop, so we do not consider the ignoreMissingJournalFiles value in the 
> call to getNextLocation(), even though missing journal files can cause 
> getNextLocation() to fail.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (AMQ-6938) KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing journal files

2018-03-25 Thread Tim Bain (JIRA)

[ 
https://issues.apache.org/jira/browse/AMQ-6938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16413351#comment-16413351
 ] 

Tim Bain commented on AMQ-6938:
---

This appears to be part of the error described in AMQ-6929, but I've pulled it 
out into a standalone issue to put the focus on the fact that we're not 
respecting the configuration provided via the ignoreMissingJournalFiles 
property, so that AMQ-6929 can focus on whatever the root cause was, if it can 
be determined.

> KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing 
> journal files
> -
>
> Key: AMQ-6938
> URL: https://issues.apache.org/jira/browse/AMQ-6938
> Project: ActiveMQ
>  Issue Type: Bug
>  Components: KahaDB
>Affects Versions: 5.13.1
>Reporter: Tim Bain
>Priority: Major
>
> It was reported on the user mailing list that a user is seeing the broker 
> fail to start due to "java.io.IOException: Could not locate data file 
> /data/kahadb/db-531.log" even though ignoreMissingJournalFiles is set to 
> true. Presumably the purpose of that option is to allow the broker to ignore 
> missing journal files (otherwise we chose poorly when naming the option and 
> then failed to document it adequately on the wiki), which means the option 
> doesn't work properly in all situations.
> Looking at the code on master as of 3/25/2018, 
> org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
> journal.getNextLocation(null) on line 677, which will throw the exception in 
> question when DataFile.getDataFile() is called. This call to 
> journal.getNextLocation() occurs outside of the while loop, whereas the 
> ignoreMissingJournalFiles value is only checked in the catch block within the 
> while loop, so we do not consider the ignoreMissingJournalFiles value in the 
> call to getNextLocation(), even though missing journal files can cause 
> getNextLocation() to fail.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (AMQ-6938) KahaDB's ignoreMissingJournalFiles argument doesn't actually ignore missing journal files

2018-03-25 Thread Tim Bain (JIRA)
Tim Bain created AMQ-6938:
-

 Summary: KahaDB's ignoreMissingJournalFiles argument doesn't 
actually ignore missing journal files
 Key: AMQ-6938
 URL: https://issues.apache.org/jira/browse/AMQ-6938
 Project: ActiveMQ
  Issue Type: Bug
  Components: KahaDB
Affects Versions: 5.13.1
Reporter: Tim Bain


It was reported on the user mailing list that a user is seeing the broker fail 
to start due to "java.io.IOException: Could not locate data file 
/data/kahadb/db-531.log" even though ignoreMissingJournalFiles is set to true. 
Presumably the purpose of that option is to allow the broker to ignore missing 
journal files (otherwise we chose poorly when naming the option and then failed 
to document it adequately on the wiki), which means the option doesn't work 
properly in all situations.

Looking at the code on master as of 3/25/2018, 
org.apache.activemq.store.kahadb.MessageDatabase.recover() calls 
journal.getNextLocation(null) on line 677, which will throw the exception in 
question when DataFile.getDataFile() is called. This call to 
journal.getNextLocation() occurs outside of the while loop, whereas the 
ignoreMissingJournalFiles value is only checked in the catch block within the 
while loop, so we do not consider the ignoreMissingJournalFiles value in the 
call to getNextLocation(), even though missing journal files can cause 
getNextLocation() to fail.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)