Async Redis cannot use hiredis pack_command

[jinohkang-theori]

PR #2570 introduced hiredis pack_command mode for sync Redis, but the change never landed for the async counterpart:

redis-py/redis/asyncio/connection.py

Lines 827 to 843 in d762623

	def pack_command(self, *args: EncodableT) -> List[bytes]:
	"""Pack a series of arguments into the Redis protocol"""
	output = []
	# the client might have included 1 or more literal arguments in
	# the command name, e.g., 'CONFIG GET'. The Redis server expects these
	# arguments to be sent separately, so split the first argument
	# manually. These arguments should be bytestrings so that they are
	# not encoded.
	assert not isinstance(args[0], float)
	if isinstance(args[0], str):
	args = tuple(args[0].encode().split()) + args[1:]
	elif b" " in args[0]:
	args = tuple(args[0].split()) + args[1:]
	buff = SYM_EMPTY.join((SYM_STAR, str(len(args)).encode(), SYM_CRLF))
	buffer_cutoff = self._buffer_cutoff

redis-py/redis/asyncio/connection.py

Lines 873 to 881 in d762623

	def pack_commands(self, commands: Iterable[Iterable[EncodableT]]) -> List[bytes]:
	"""Pack multiple commands into the Redis protocol"""
	output: List[bytes] = []
	pieces: List[bytes] = []
	buffer_length = 0
	buffer_cutoff = self._buffer_cutoff
	for cmd in commands:
	for chunk in self.pack_command(*cmd):

would love to see pack_command support landed for asyncio Redis too.

[petyaslavova]

Hey @jinohkang-theori, thanks for bringing this to our attention! We will have a look at it.

[sakshichitnis27]

Hi, I’d like to work on this issue.

I confirmed that the synchronous connection selects HiredisRespSerializer when hiredis.pack_command is available, while redis.asyncio still uses the Python command-packing implementation.

I plan to bring the same packer selection to asyncio connections, preserve the pure-Python fallback and existing memoryview behaviour, and add focused tests for both hiredis-enabled and fallback paths.

Would you prefer sharing the existing serializer classes between the sync and async connection modules, or keeping the asyncio implementation separate to avoid additional import coupling?