[ 
https://issues.apache.org/jira/browse/ARTEMIS-4065?focusedWorklogId=832840&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-832840
 ]

ASF GitHub Bot logged work on ARTEMIS-4065:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 12/Dec/22 18:05
            Start Date: 12/Dec/22 18:05
    Worklog Time Spent: 10m 
      Work Description: clebertsuconic commented on code in PR #4265:
URL: https://github.com/apache/activemq-artemis/pull/4265#discussion_r1046200326


##########
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageSubscriptionCounterImpl.java:
##########
@@ -465,4 +419,17 @@ public void addAndGet(int count, long persistentSize) {
          SIZE_UPDATER.addAndGet(this, persistentSize);
       }
    }
+
+   @Override
+   public PageSubscriptionCounter setSubscription(PageSubscription 
subscription) {
+      this.subscription = subscription;
+
+      if (subscription == null) {
+         this.pageExecutor = null;
+      } else {
+         this.pageExecutor = subscription.getPagingStore().getExecutor();
+         assert pageExecutor != null;

Review Comment:
   not any more... the previous version did.. removing it.. thanks





Issue Time Tracking
-------------------

    Worklog Id:     (was: 832840)
    Time Spent: 1h 40m  (was: 1.5h)

> Option to use non persistent counters in paging. Rebuild them upon start if 
> persistence is disabled on them
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: ARTEMIS-4065
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4065
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Clebert Suconic
>            Priority: Major
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> Instead of storing records on journal for counting how many records there are 
> in paging, the system should instead just swipe the paging system in parallel 
> with processing data.
> The changes I'm making will take a snapshot of the current records of paging, 
> and then it will read all the pages to rebuild the counters.
> On tests I am making from a real data server, a system that had a lot of 
> pages (700) needed less than 1 minute to rebuild the counters, and the 
> messages were available to be delivered while the swipe was being done.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to