codelipenghui commented on a change in pull request #5570: Transaction log 
implemention
URL: https://github.com/apache/pulsar/pull/5570#discussion_r346198793
 
 

 ##########
 File path: 
pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/impl/ManagedLedgerTransactionReaderImpl.java
 ##########
 @@ -0,0 +1,212 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.transaction.coordinator.impl;
+
+import io.netty.buffer.ByteBuf;
+
+import java.util.ArrayList;
+import java.util.List;
+import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.ConcurrentMap;
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.atomic.AtomicLong;
+
+import org.apache.bookkeeper.mledger.AsyncCallbacks.ReadEntriesCallback;
+import org.apache.bookkeeper.mledger.Entry;
+import org.apache.bookkeeper.mledger.ManagedLedgerConfig;
+import org.apache.bookkeeper.mledger.ManagedLedgerException;
+import org.apache.bookkeeper.mledger.ManagedLedgerFactory;
+import org.apache.bookkeeper.mledger.ReadOnlyCursor;
+import org.apache.bookkeeper.mledger.impl.PositionImpl;
+import org.apache.pulsar.common.api.proto.PulsarApi.Subscription;
+import org.apache.pulsar.common.api.proto.PulsarApi.TransactionMetadataEntry;
+import org.apache.pulsar.common.api.proto.PulsarApi.TxnStatus;
+import org.apache.pulsar.common.util.protobuf.ByteBufCodedInputStream;
+import org.apache.pulsar.transaction.coordinator.TransactionMetadataStore;
+import org.apache.pulsar.transaction.coordinator.TxnMeta;
+import org.apache.pulsar.transaction.coordinator.TxnSubscription;
+import 
org.apache.pulsar.transaction.coordinator.exceptions.InvalidTxnStatusException;
+import org.apache.pulsar.transaction.impl.common.TxnID;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+class ManagedLedgerTransactionReaderImpl implements
+        ManagedLedgerTransactionMetadataStore.ManagedLedgerTransactionReader {
+
+    private static final Logger log = 
LoggerFactory.getLogger(ManagedLedgerTransactionReaderImpl.class);
+
+    private ConcurrentMap<TxnID, TxnMeta> txnMetaMap = new 
ConcurrentHashMap<>();
+
+    private AtomicLong sequenceId = new 
AtomicLong(ManagedLedgerTransactionMetadataStore.TC_ID_NOT_USED);
+
+    private final ReadOnlyCursor readOnlyCursor;
+
+    public ManagedLedgerTransactionReaderImpl(String tcId, 
ManagedLedgerFactory managedLedgerFactory) throws Exception {
+        this.readOnlyCursor = managedLedgerFactory
+                .openReadOnlyCursor(tcId,
+                        PositionImpl.earliest, new ManagedLedgerConfig());
+    }
+
+    private static List<TxnSubscription> 
subscriptionToTxnSubscription(List<Subscription> subscriptions) {
+        List<TxnSubscription> txnSubscriptions = new 
ArrayList<>(subscriptions.size());
+        for (int i = 0; i < subscriptions.size(); i++) {
+            txnSubscriptions
+                    .add(new TxnSubscription(subscriptions.get(i).getTopic(), 
subscriptions.get(i).getSubscription()));
+        }
+        return txnSubscriptions;
+    }
+
+
+    @Override
+    public TxnMeta getTxnMeta(TxnID txnID) {
+        return txnMetaMap.get(txnID);
+    }
+
+    @Override
+    public Long readSequenceId() {
+        return sequenceId.get();
+    }
+
+    @Override
+    public void addNewTxn(TxnMeta txnMeta) {
+        txnMetaMap.put(txnMeta.id(), txnMeta);
+    }
+
+    @Override
+    public TxnStatus getTxnStatus(TxnID txnID) {
+        return txnMetaMap.get(txnID).status();
+    }
+
+    @Override
+    public CompletableFuture<Void> init(TransactionMetadataStore 
transactionMetadataStore) {
+        CountDownLatch countDownLatch = new CountDownLatch(1);
+        countDownLatch.countDown();
+        CompletableFuture<Void> completableFuture = new CompletableFuture<>();
+        
transactionMetadataStore.updateMetadataStoreState(TransactionMetadataStore.State.INITIALIZING);
+        readOnlyCursor
+                .asyncReadEntries(100,
+                        new ReaderReadEntriesCallback(countDownLatch, 
txnMetaMap, sequenceId,
+                                transactionMetadataStore, completableFuture), 
System.nanoTime());
+        return completableFuture;
+    }
+
+    @Override
+    public void close() throws ManagedLedgerException, InterruptedException {
+        txnMetaMap.clear();
+        readOnlyCursor.close();
+    }
+
+    class ReaderReadEntriesCallback implements ReadEntriesCallback {
+
+        private final CountDownLatch originalCountDownLatch;
+        private final CountDownLatch currentCountDownLatch = new 
CountDownLatch(1);
+        private final ConcurrentMap<TxnID, TxnMeta> txnMetaMap;
+        private AtomicLong sequenceId;
+        private TransactionMetadataStore transactionMetadataStore;
+        private CompletableFuture<Void> completableFuture;
+
+        ReaderReadEntriesCallback(CountDownLatch originalCountDownLatch,
+                                  ConcurrentMap<TxnID, TxnMeta> txnMetaMap,
+                                  AtomicLong sequenceId,
+                                  TransactionMetadataStore 
transactionMetadataStore,
+                                  CompletableFuture<Void> completableFuture) {
+            this.originalCountDownLatch = originalCountDownLatch;
+            this.txnMetaMap = txnMetaMap;
+            this.sequenceId = sequenceId;
+            this.transactionMetadataStore = transactionMetadataStore;
+            this.completableFuture = completableFuture;
+        }
+
+        @Override
+        public void readEntriesComplete(List<Entry> entries, Object ctx) {
+            try {
+                if (readOnlyCursor.hasMoreEntries()) {
+                    readOnlyCursor.asyncReadEntries(100,
+                            new 
ReaderReadEntriesCallback(currentCountDownLatch, txnMetaMap,
+                                    sequenceId, transactionMetadataStore, 
completableFuture), System.nanoTime());
+                }
+                originalCountDownLatch.await();
+                for (int i = 0; i < entries.size(); i++) {
+                    ByteBuf buffer = entries.get(i).getDataBuffer();
+                    ByteBufCodedInputStream stream = 
ByteBufCodedInputStream.get(buffer);
+                    TransactionMetadataEntry.Builder 
transactionMetadataEntryBuilder =
+                            TransactionMetadataEntry.newBuilder();
+                    TransactionMetadataEntry transactionMetadataEntry =
+                            transactionMetadataEntryBuilder.mergeFrom(stream, 
null).build();
+                    TxnID txnID = new 
TxnID(transactionMetadataEntry.getTxnidMostBits(),
+                            transactionMetadataEntry.getTxnidLeastBits());
+                    switch (transactionMetadataEntry.getMetadataOp()) {
+                        case NEW:
+                            if (sequenceId.get() < 
transactionMetadataEntry.getTxnidLeastBits()) {
+                                
sequenceId.set(transactionMetadataEntry.getTxnidLeastBits());
+                            }
+                            txnMetaMap.put(txnID, new TxnMetaImpl(txnID));
+                            transactionMetadataEntryBuilder.recycle();
+                            stream.recycle();
+                            break;
+                        case ADD_PARTITION:
+                            txnMetaMap.get(txnID)
+                                    
.addProducedPartitions(transactionMetadataEntry.getPartitionsList());
+                            transactionMetadataEntryBuilder.recycle();
+                            stream.recycle();
+                            break;
+                        case ADD_SUBSCRIPTION:
+                            txnMetaMap.get(txnID)
+                                    .addTxnSubscription(
+                                            subscriptionToTxnSubscription
+                                                    
(transactionMetadataEntry.getSubscriptionsList()));
+                            transactionMetadataEntryBuilder.recycle();
+                            stream.recycle();
+                            break;
+                        case UPDATE:
+                            txnMetaMap.get(txnID)
+                                    
.updateTxnStatus(transactionMetadataEntry.getNewStatus(),
+                                            
transactionMetadataEntry.getExpectedStatus());
+                            transactionMetadataEntryBuilder.recycle();
+                            stream.recycle();
+                            break;
+                        default:
+                            throw new InvalidTxnStatusException("Transaction `"
+                                    + txnID + "` load bad metadata operation 
from transaction log ");
+                    }
+                    entries.get(i).release();
+                }
+                currentCountDownLatch.countDown();
+                if (!readOnlyCursor.hasMoreEntries()) {
+                    originalCountDownLatch.await();
 
 Review comment:
   read only cursor can be closed while have no available messages?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to