Bug: XRay SDK doesn't support pg8000 native driver
@leandrodamascena ya está trabajando en esto.
Desde el 20/1/2025.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Empieza leyendo aws_xray_sdk/ext/pg8000/patch.py, especialmente el tratamiento existente de pg8000.connect, y compáralo después con pg8000.native.Connection y su punto de entrada run descrito en el issue. Se considera terminado cuando las conexiones nativas y las llamadas a run producen subsegmentos SQL de X-Ray capturados, como en la ruta de la Standard API.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
pg8000 comes with two APIs, the pg8000 native API and the DB-API 2.0 Standard API (PEP-0249) and the XRay SDK fails to capture SQL segments when using the pg8000 native API.
Explanation, code example and output
When working with the pg8000 native API, customers initialize a connection using pg8000.native.Connection and xray is unable to intercept and wrapper this because in this line it always considers pg8000.connect and not pg8000.native.Connection. Also, when using pg8000 native API the connection instance doesn't have cursor, all the queries are executed directly using run command.
Check this code example:
from urllib.parse import quote_plus
import pg8000.native
from aws_xray_sdk.core import xray_recorder
from aws_xray_sdk.core import patch_all
# Initialize X-Ray
xray_recorder.configure(service='MyPostgresApp')
patch_all()
@xray_recorder.capture('connect_to_db')
def connect_to_db():
try:
host = "MYHOST"
user = "MYUSER"
database = "MYDATABASE"
password = "MYPASSWORD"
conn = pg8000.native.Connection(
database=database,
user=user,
password=password,
host=host,
port=5432,
ssl_context=True
)
return conn
except Exception as e:
print(f"Error connecting to database: {e}")
return None
@xray_recorder.capture('query_db')
def query_db(conn):
try:
results = conn.run("SELECT * FROM public.todos LIMIT 5")
return results
except Exception as e:
print(f"Error querying database: {e}")
return None
def main():
with xray_recorder.in_segment('main_function'):
conn = connect_to_db()
if conn:
results = query_db(conn)
if results:
print("Query results:", results)
conn.close()
if __name__ == "__main__":
main()
This creates the json below and we see that there is no run subsegment in the query_db segment because XRAY cannot patch this driver when using native Connection.
{
"id":"caeb26df6fe242ed",
"name":"main_function",
"start_time":1736462563.884077,
"in_progress":false,
"aws":{
"xray":{
"sdk":"X-Ray for Python",
"sdk_version":"2.14.0"
}
},
"subsegments":[
{
"id":"981eb724c5369158",
"name":"connect_to_db",
"start_time":1736462563.884105,
"parent_id":"caeb26df6fe242ed",
"in_progress":false,
"trace_id":"1-678050e3-fbec3813f4280021b9069ff5",
"type":"subsegment",
"namespace":"local",
"end_time":1736462564.930994
},
{
"id":"eb5a8872a770eacd",
"name":"query_db",
"start_time":1736462564.931169,
"parent_id":"caeb26df6fe242ed",
"in_progress":false,
"trace_id":"1-678050e3-fbec3813f4280021b9069ff5",
"type":"subsegment",
"namespace":"local",
"end_time":1736462565.3793
}
],
"trace_id":"1-678050e3-fbec3813f4280021b9069ff5",
"service":{
"runtime":"CPython",
"runtime_version":"3.12.8"
},
"end_time":1736462565.3813171
}
Solution
I created a small PoC locally and made it work by adding a new method to wrap native.Connection and creating a new class called XRayTracedNative to intercept the run method and then be able to create the subsegment.
*the names are just suggestions, you can choose whatever name you want.
Please let me know if you are interested in a PR to implement this and I can work.
- Lenguaje dominante
- Python
- Estrellas
- 339
- Forks
- 148
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de aws/aws-xray-sdk-python
-
psycopg connection context manager drops X-Ray tracing wrapper, so cursor execute calls are not tracedPosiblemente ocupada @reginaldalfret la tomó hace 17 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
aws/aws-xray-sdk-python#490 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 1/100
aws/aws-xray-sdk-python#460 ·
-
Next releaseAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
aws/aws-xray-sdk-python#457 · 1 reacción ·
-
mysql-connector 9.2.0 - TypeError: 'MySQLConnection' object is not iterablePosiblemente ocupada @reginaldalfret la tomó hace 17 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
aws/aws-xray-sdk-python#453 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
aws/aws-xray-sdk-python#452 · 1 reacción ·
Todos los issues de aws/aws-xray-sdk-python
Issues similares
-
🐛 Bug: Calendar work items cannot be opened on mobilePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 2 días
-
Status: Triage Type: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
-
Objects.List treats versions=false (and similar flags) as truePosiblemente ocupada @andaaron la tomó hoy. Abiertopriority: p2 type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
googleapis/storage-testbench#829 ·
Los mantenedores suelen responder en 2 días
-
[Bug]: memory provider treats 1:1 DMs as shared surfaces, so DM turns never receive reviewed recordsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
rlaope/oh-my-hermes#2088 ·
Los mantenedores suelen responder en 1 día