Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,15 @@ All notable changes to this project will be documented in this file.

- Support configuring compression for OTEL ([#70]).
- Improve tracing details by adding a `tower_http::trace::TraceLayer` that creates spans for every HTTP request ([#71]).

### Changed

- Improve tracing for running queries on Trino, adding spans for the request to Trino and parsing ([#71]).
- Improve performance by using [`serde_json::value::RawValue`](https://docs.rs/serde_json/latest/serde_json/value/struct.RawValue.html) for the `data` and `columns` attributes to avoid unneeded deserialization and serialization of them ([#73]).

[#70]: https://github.com/stackabletech/trino-lb/pull/70
[#71]: https://github.com/stackabletech/trino-lb/pull/71
[#73]: https://github.com/stackabletech/trino-lb/pull/73

## [0.4.1] - 2025-03-03

Expand Down
5 changes: 3 additions & 2 deletions trino-lb-core/src/trino_api.rs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ use std::{

use prusto::{QueryError, Warning};
use serde::{Deserialize, Serialize};
use serde_json::value::RawValue;
use snafu::{ResultExt, Snafu};
use tracing::instrument;
use url::Url;
Expand Down Expand Up @@ -42,8 +43,8 @@ pub struct TrinoQueryApiResponse {
pub info_uri: String,
pub partial_cancel_uri: Option<String>,

pub columns: Option<serde_json::Value>,
pub data: Option<serde_json::Value>,
pub columns: Option<Box<RawValue>>,
pub data: Option<Box<RawValue>>,

pub error: Option<QueryError>,
pub warnings: Vec<Warning>,
Expand Down
Loading