Bug 1518851 - [GSS](6.4.z) Add CriticalAnalyzer to TimedBuffer operations follow up to BZ1487313
Summary: [GSS](6.4.z) Add CriticalAnalyzer to TimedBuffer operations follow up to BZ14...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: JMS
Version: 6.4.18
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: CR1
: EAP 6.4.19
Assignee: jboss-set
QA Contact: Peter Mackay
URL:
Whiteboard:
Depends On: 1487313
Blocks: eap6419-payload 1498190
TreeView+ depends on / blocked
 
Reported: 2017-11-29 16:18 UTC by Tom Ross
Modified: 2021-03-11 16:26 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-04-16 11:03:27 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Apache JIRA ARTEMIS-1531 0 None None None 2017-12-01 05:31:30 UTC
Red Hat Issue Tracker HORNETQ-1566 0 Major Open Add CriticalAnalyzer to TimedBuffer operations 2018-08-14 02:17:37 UTC
Red Hat Issue Tracker JBEAP-13890 0 Critical New [GSS](7.2.0) Add CriticalAnalyzer to TimedBuffer operations 2018-08-14 02:17:37 UTC
Red Hat Issue Tracker JBEAP-13891 0 Critical New [GSS](7.0.z) Add CriticalAnalyzer to TimedBuffer operations 2018-08-14 02:17:37 UTC
Red Hat Issue Tracker JBEAP-13892 0 Critical Verified [GSS](7.1.z) Add CriticalAnalyzer to TimedBuffer operations 2018-08-14 02:17:37 UTC

Description Tom Ross 2017-11-29 16:18:40 UTC
The stop() method on TimedBuffer class should be added to critical path of CriticalAnalyzer and it should be synchronised.

Comment 2 Clebert Suconic 2017-11-30 15:03:01 UTC
This is being applied in Artemis: https://github.com/apache/activemq-artemis/pull/1681


It is already applied in hornetq: 

commit d4c29e29c472ba90f313c774cd0f96f62006ff12
Author: Clebert Suconic <clebertsuconic>
Date:   Wed Nov 29 14:58:07 2017 -0500

    BZ-1518851 & BZ-1487313 Fixing TimedBuffer and File Sync
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1518851
    https://bugzilla.redhat.com/show_bug.cgi?id=1487313

Comment 5 Jiří Bílek 2018-01-17 09:18:31 UTC
Verified with EAP 6.4.19.CP.CR2

Note: Included test is not executed automatically with any profile. Here is PR https://github.com/hornetq/hornetq/pull/2126 with fix.


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