<!-- ATTENTION: Verify that issue you are about to submit is not a duplicate. --> ## General Summary Расчет bucket_id по одному и тому же значению `DECIMAL` попадает в разные бакеты в зависимости от того, передано оно bind-параметром или литералом в тексте запроса. ## Expected Behavior Значение `DECIMAL`, используемое как ключ распределения, должно давать один и тот же `bucket_id` независимо от способа передачи в запрос (bind-параметр или литерал). ## Current Behavior Bind-параметр и литерал разбираются разными путями, с разными пределами точности и в результате одно и то же число даёт разные `bucket_id`. ## Steps to Reproduce ``` import asyncio import os from decimal import Decimal import asyncpg TABLE = "bind_vs_literal" # количество значащих цифр -> значение VALUES = [ # максимум в rust_decimal https://docs.rs/rust_decimal/latest/rust_decimal/#comparison-to-other-decimal-implementations ("29", Decimal("1.1234567890123456789012345678")), # пикодата может ограничивать decimal 29-ю значащими цифрами перед вычислением bucket_id # т.к. начиная с этого значения, результат вычислений через rust_decimal будет одинаковый ("30", Decimal("1.12345678901234567890123456789")), ("31", Decimal("1.123456789012345678901234567896")), # максимум в тарантул ("38", Decimal("1.1234567890123456789012345678901234567")), # тарантул сначала ограничивает decimal 38 значащими цифрами, после чего уже считается bucket_id. # т.е. начиная с этого значения, результат вычислений через tarantool::decimal будет одинаковый ("39", Decimal("1.12345678901234567890123456789012345678")), ("40", Decimal("1.123456789012345678901234567890123456786")), ] # вставить значение и вернуть bucket_id, вычисленный пикодатой async def _insert_and_read(conn: asyncpg.Connection, row_id: int, query: str, *args) -> int | str: try: await conn.execute(query, *args) row = await conn.fetchrow(f'SELECT bucket_id FROM "{TABLE}" WHERE id = $1', row_id) return row['bucket_id'] except asyncpg.PostgresError as e: return f"ошибка выполнения: {e}" async def main() -> None: dsn = os.environ.get("PICODATA_TEST_CONN", "postgresql://admin:T0psecret@localhost:5433") conn = await asyncpg.connect(dsn) try: await conn.execute(f'DROP TABLE IF EXISTS "{TABLE}";') await conn.execute( f'CREATE TABLE "{TABLE}" (id INTEGER PRIMARY KEY, d DECIMAL NOT NULL) ' f'USING memtx DISTRIBUTED BY (d) IN TIER "default" OPTION (TIMEOUT = 3.0);' ) bind_query = f'INSERT INTO "{TABLE}" VALUES ($1, $2)' for i, (label, value) in enumerate(VALUES): await conn.execute(f'DELETE FROM "{TABLE}";') # вставка с bind decimal'ом bind_id = 2 * i + 1 bucket_from_bind = await _insert_and_read(conn, bind_id, bind_query, bind_id, value) # вставка с литералами literal_id = 2 * i + 2 bucket_from_literal = await _insert_and_read( conn, literal_id, f'INSERT INTO "{TABLE}" VALUES ({literal_id}, {value})', ) # ожидается, что bucket_id будут равны print( f"(количество значащих цифр {label}) значение: {value}\n" f"bucket при bind decimal: {bucket_from_bind}\n" f"bucket при literal: {bucket_from_literal}" ) finally: await conn.execute(f'DROP TABLE IF EXISTS "{TABLE}";') await conn.close() asyncio.run(main()) ``` результаты: ``` (количество значащих цифр 29) значение: 1.1234567890123456789012345678 bucket при bind decimal: 2405 bucket при literal: 2405 (количество значащих цифр 30) значение: 1.12345678901234567890123456789 bucket при bind decimal: 2387 bucket при literal: 55 (количество значащих цифр 31) значение: 1.123456789012345678901234567896 bucket при bind decimal: 2387 bucket при literal: 1872 (количество значащих цифр 38) значение: 1.1234567890123456789012345678901234567 bucket при bind decimal: 2387 bucket при literal: 788 (количество значащих цифр 39) значение: 1.12345678901234567890123456789012345678 bucket при bind decimal: 2387 bucket при literal: 1263 (количество значащих цифр 40) значение: 1.123456789012345678901234567890123456786 bucket при bind decimal: 2387 bucket при literal: 1263 asya@asya-ThinkPad-P15v-Ge ``` ## Possible Solution Судя по всему, bind параметры проходят конвертацию в rust_decimal::Decimal, у которого ограничения отличающиеся от тарантула, в то время как литерал парсится напрямую в tarantool::decimal::Decimal https://docs.rs/rust_decimal/latest/rust_decimal/#comparison-to-other-decimal-implementations ``` The downside of this approach however is that the maximum number of significant digits that can be represented is roughly 28 base-10 digits (29 in some cases). ``` <!-- Used by GitLab to automatically set a label for created ticket. Do not touch unless necessary! -->