perf improve (#367)

* retire traces

* compact ticket attempt

* change L config in tiny

* data decode and slice

* cache trap

* avoid runtime private(set) check

* cache ring context

* add decode lenghth check

* improve ring context

* avoid overflow

* add checks

* improve benchmark ci

* update benchmark ci

* ignore benchmark files

* revert ticket changes (test vector not updated yet)

* fix ring context

Author: qiweiii

.github/workflows/benchmark-pr.yml

@@ -5,10 +5,6 @@ on:
   pull_request:
     branches: [ master ]
 
-concurrency:
-  group: ${{ github.workflow }}-${{ github.ref }}
-  cancel-in-progress: true
-
 env:
   BENCHMARK_DISABLE_JEMALLOC: true
 
@@ -124,35 +120,18 @@ jobs:
 
       - name: Compare PR and master
         if: ${{ env.hasBenchmark == '1' }}
-        id: benchmark
         run: |
           cd JAMTests
-          echo "# 📊 Benchmark Results"
-          echo "Generated on: $(date)"
-          echo ""
-          echo "exitStatus=1" >> $GITHUB_ENV
+          echo "# 📊 Benchmark Results" >> $GITHUB_STEP_SUMMARY
+          echo "Generated on: $(date)" >> $GITHUB_STEP_SUMMARY
+          echo "" >> $GITHUB_STEP_SUMMARY
 
-          # Check if master baseline exists for comparison
           if [ -f ".benchmarkBaselines/BenchmarkTestVectors/master/results.json" ]; then
-            echo "## 🔄 Comparison with Master Branch"
-            echo ""
-            swift package benchmark baseline check master pull_request --format markdown
-            echo "exitStatus=0" >> $GITHUB_ENV
+            swift package benchmark baseline compare master pull_request --no-progress --format markdown >> $GITHUB_STEP_SUMMARY 2>&1 || true
+            echo "" >> $GITHUB_STEP_SUMMARY
+            echo "## Threshold Check" >> $GITHUB_STEP_SUMMARY
+            swift package benchmark baseline check master pull_request --format markdown >> $GITHUB_STEP_SUMMARY 2>&1 || true
           else
-            echo "## 🆕 First Benchmark Run"
-            echo ""
-            echo "⚠️ Master baseline not found. This appears to be the first benchmark run."
-            echo "The following results will become the baseline for future comparisons:"
-            echo ""
-
-            # Run benchmarks to generate results for first time
-            echo "### Benchmark Results:"
-            swift package benchmark --format markdown 2>&1 || true
-            echo "exitStatus=0" >> $GITHUB_ENV
+            echo "⚠️ Master baseline not found. Running benchmarks for this PR:" >> $GITHUB_STEP_SUMMARY
+            swift package benchmark --format markdown >> $GITHUB_STEP_SUMMARY 2>&1 || true
           fi
-        continue-on-error: true
-
-      - name: Exit with correct status
-        if: ${{ env.hasBenchmark == '1' }}
-        run: |
-          exit ${{ env.exitStatus }}

.gitignore

@@ -30,3 +30,5 @@ Tools/openrpc.json
 tmp
 
 *.pvm
+
+.benchmarkBaselines

Blockchain/Sources/Blockchain/Config/ProtocolConfig+Preset.swift

@@ -121,7 +121,7 @@ extension Ref where T == ProtocolConfig {
         maxWorkItems: 16,
         maxDepsInWorkReport: 8,
         maxTicketsPerExtrinsic: 3,
-        maxLookupAnchorAge: 14400,
+        maxLookupAnchorAge: 24,
         transferMemoSize: 128,
         ticketEntriesPerValidator: 3,
         maxAuthorizationsPoolItems: 8,

Codec/Sources/Codec/IntegerCodec.swift

@@ -12,6 +12,7 @@ extension Collection<UInt8> where SubSequence == Self {
         IntegerCodec.decode { self.next() }
     }
 
+    // NOTE: for data, use decodeUInt* methods in Data+Utils.swift for better performance
     public mutating func decode<T: UnsignedInteger>(length: Int) -> T? {
         guard length > 0, length <= count else { return nil }