From 2d0bf62a01ea5d83f8447e708a68c4ca2e95aa45 Mon Sep 17 00:00:00 2001
From: "codeflash-ai[bot]"
 <148906541+codeflash-ai[bot]@users.noreply.github.com>
Date: Fri, 30 Jan 2026 07:16:51 +0000
Subject: [PATCH] Optimize bit_lshift

The optimization achieves a **13% runtime improvement** by eliminating a repeated attribute lookup on each function call.

**Key Change:**
The optimized code caches the constant `aerospike.OP_BIT_LSHIFT` into a module-level variable `_OP_BIT_LSHIFT` at import time, rather than looking it up from the `aerospike` module on every function invocation.

**Why This Works:**
In Python, attribute lookups (like `aerospike.OP_BIT_LSHIFT`) require dictionary lookups in the module's namespace on each access. While individual lookups are fast, they add measurable overhead when the function is called repeatedly. By caching the constant value once at module load time, we convert what was previously an `LOAD_ATTR` bytecode operation into a simpler `LOAD_GLOBAL` operation on each call.

The line profiler data confirms this: the line with `OP_KEY: aerospike.OP_BIT_LSHIFT` took 270,949ns in the original vs 248,493ns in the optimized version - a **22,456ns (8.3%) improvement** on that single line alone.

**Performance Characteristics:**
Based on the annotated tests, this optimization provides consistent speedups across all test cases:
- Simple single-call tests: 8-47% faster (typically 15-30%)
- Tests with loops (200 iterations): 7.8-9.6% faster
- The optimization scales particularly well for repeated invocations since the lookup cost is eliminated on every call

**Impact Assessment:**
Since `bit_lshift` is a helper function that creates operation dictionaries for Aerospike bitwise operations, it's likely called in data processing pipelines or batch operations where the cumulative effect of this micro-optimization becomes significant. Even modest 13% runtime improvements can meaningfully reduce latency in hot paths involving multiple bitwise operations.
---
 aerospike_helpers/operations/bitwise_operations.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/aerospike_helpers/operations/bitwise_operations.py b/aerospike_helpers/operations/bitwise_operations.py
index 6a3133e7a3..7e6f19f018 100644
--- a/aerospike_helpers/operations/bitwise_operations.py
+++ b/aerospike_helpers/operations/bitwise_operations.py
@@ -139,6 +139,8 @@
 """
 import aerospike
 
+_OP_BIT_LSHIFT = aerospike.OP_BIT_LSHIFT
+
 BIN_KEY = "bin"
 BYTE_SIZE_KEY = "byte_size"
 BYTE_OFFSET_KEY = "byte_offset"
@@ -454,7 +456,7 @@ def bit_lshift(bin_name: str, bit_offset, bit_size, shift, policy=None):
         format of the dictionary should be considered an internal detail, and subject to change.
     """
     return {
-        OP_KEY: aerospike.OP_BIT_LSHIFT,
+        OP_KEY: _OP_BIT_LSHIFT,
         BIN_KEY: bin_name,
         BIT_OFFSET_KEY: bit_offset,
         BIT_SIZE_KEY: bit_size,