[GitHub] [activemq-nms-amqp] Havret edited a comment on issue #28: NO-JIRA: Extend logging

2019-09-20 Thread GitBox
Havret edited a comment on issue #28: NO-JIRA: Extend logging URL: https://github.com/apache/activemq-nms-amqp/pull/28#issuecomment-533690118 I'm not a Java expert, but what I remember from Uncle Bob's Clean Code I read years ago, you shouldn't use checked exception as they violate open

[GitHub] [activemq-nms-amqp] Havret commented on a change in pull request #33: NO-JIRA: Update docs

2019-09-20 Thread GitBox
Havret commented on a change in pull request #33: NO-JIRA: Update docs URL: https://github.com/apache/activemq-nms-amqp/pull/33#discussion_r326775649 ## File path: docs/configuration.md ## @@ -0,0 +1,21 @@ +# Client configuration +This file details various configuration

[GitHub] [activemq-nms-amqp] Havret commented on a change in pull request #33: NO-JIRA: Update docs

2019-09-20 Thread GitBox
Havret commented on a change in pull request #33: NO-JIRA: Update docs URL: https://github.com/apache/activemq-nms-amqp/pull/33#discussion_r326775649 ## File path: docs/configuration.md ## @@ -0,0 +1,21 @@ +# Client configuration +This file details various configuration

[GitHub] [activemq-nms-amqp] Havret edited a comment on issue #28: NO-JIRA: Extend logging

2019-09-20 Thread GitBox
Havret edited a comment on issue #28: NO-JIRA: Extend logging URL: https://github.com/apache/activemq-nms-amqp/pull/28#issuecomment-533690118 I'm not a Java expert, but what I remember from Uncle Bob's Clean Code I read years ago, you shouldn't use checked exception as they violate open

[GitHub] [activemq-nms-amqp] Havret edited a comment on issue #28: NO-JIRA: Extend logging

2019-09-20 Thread GitBox
Havret edited a comment on issue #28: NO-JIRA: Extend logging URL: https://github.com/apache/activemq-nms-amqp/pull/28#issuecomment-533690118 I'm not a Java expert, but what I remember from Uncle Bob's Clean Code I read years ago, you shouldn't use checked exception as they violate open

[GitHub] [activemq-nms-amqp] cjwmorgan-sol commented on issue #28: NO-JIRA: Extend logging

2019-09-20 Thread GitBox
cjwmorgan-sol commented on issue #28: NO-JIRA: Extend logging URL: https://github.com/apache/activemq-nms-amqp/pull/28#issuecomment-533730425 I find having the provider intentionally hiding potential unintended errors at the cost of a log for a single scenario seems like a bad idea.

[GitHub] [activemq-nms-amqp] Havret commented on issue #28: NO-JIRA: Extend logging

2019-09-20 Thread GitBox
Havret commented on issue #28: NO-JIRA: Extend logging URL: https://github.com/apache/activemq-nms-amqp/pull/28#issuecomment-533690118 I'm not a Java expert, but what I remember from Uncle Bob's Clean Code I read years ago, you shouldn't use checked exception as they violate open closed

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 Just for completeness that's what would change (ie no additional copies with heap buffers

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 edited a comment on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 @wy96f @clebertsuconic Just for completeness that's what would change: -

[GitHub] [activemq-artemis] franz1981 closed pull request #2832: ARTEMIS-2482 Large messages could leak native ByteBuffers

2019-09-20 Thread GitBox
franz1981 closed pull request #2832: ARTEMIS-2482 Large messages could leak native ByteBuffers URL: https://github.com/apache/activemq-artemis/pull/2832 This is an automated message from the Apache Git Service. To respond

[GitHub] [activemq-website] gemmellr commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release

2019-09-20 Thread GitBox
gemmellr commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release URL: https://github.com/apache/activemq-website/pull/12#discussion_r326543695 ## File path: src/xml-reference.md ## @@ -51,6 +51,7 @@ XBean XML Reference

[GitHub] [activemq-website] gemmellr commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release

2019-09-20 Thread GitBox
gemmellr commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release URL: https://github.com/apache/activemq-website/pull/12#discussion_r326542262 ## File path: src/components/classic/download/index.md ## @@ -7,12 +7,12 @@ type: activemq5 These are the

[GitHub] [activemq-artemis] franz1981 commented on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 commented on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533591180 @wy96f @clebertsuconic This should solve the memory leak issue with the minimum amount of changes.

[GitHub] [activemq-artemis] franz1981 opened a new pull request #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 opened a new pull request #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844 It use RandomAccessFile to allow using heap buffers without additional copies and/or leaks of direct buffers, as

[GitHub] [activemq-website] jbonofre commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release

2019-09-20 Thread GitBox
jbonofre commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release URL: https://github.com/apache/activemq-website/pull/12#discussion_r326604413 ## File path: src/xml-reference.md ## @@ -51,6 +51,7 @@ XBean XML Reference

[GitHub] [activemq-website] jbonofre commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release

2019-09-20 Thread GitBox
jbonofre commented on a change in pull request #12: Publish ActiveMQ 5.15.10 release URL: https://github.com/apache/activemq-website/pull/12#discussion_r326604234 ## File path: src/components/classic/download/index.md ## @@ -7,12 +7,12 @@ type: activemq5 These are the

[GitHub] [activemq-website] asfgit merged pull request #12: Publish ActiveMQ 5.15.10 release

2019-09-20 Thread GitBox
asfgit merged pull request #12: Publish ActiveMQ 5.15.10 release URL: https://github.com/apache/activemq-website/pull/12 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [activemq-website] asfgit merged pull request #12: Publish ActiveMQ 5.15.10 release

2019-09-20 Thread GitBox
asfgit merged pull request #12: Publish ActiveMQ 5.15.10 release URL: https://github.com/apache/activemq-website/pull/12 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [activemq-artemis] franz1981 commented on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers

2019-09-20 Thread GitBox
franz1981 commented on issue #2844: ARTEMIS-1811 NIO Seq File should use RandomAccessFile with heap buffers URL: https://github.com/apache/activemq-artemis/pull/2844#issuecomment-533606931 Just for completeness that's what would change (ie no additional copies with heap buffers anymore):

[GitHub] [activemq-artemis] clebertsuconic commented on issue #2843: ARTEMIS-2496 Revert catch up with zero-copy, as it's causing issues i…

2019-09-20 Thread GitBox
clebertsuconic commented on issue #2843: ARTEMIS-2496 Revert catch up with zero-copy, as it's causing issues i… URL: https://github.com/apache/activemq-artemis/pull/2843#issuecomment-533541903 @wy96f is this really a feature you really would like to have? - as we moved through this