Skip to content

Low readability of debug log in BigQuery ReadRowsHelper #25409

@ebyhr

Description

@ebyhr

We shouldn't log the entire ReadRowsResponse object:

log.debug("ReadRowsResponse from BigQuery: %s", response);

2025-03-25T15:37:20.578+0900	DEBUG	bigquery-bigquery-5	io.trino.plugin.bigquery.ReadRowsHelper	ReadRowsResponse from BigQuery: stats {
  progress {
    at_response_end: 0.001469222130253911
  }
}
arrow_record_batch {
  serialized_record_batch: "\377\377\377\377\360\a\000\000\024\000\000\000\000\000\000\000\f\000\030\000\006\000\005\000\b\000\f\000\f\000\000\000\000\003\004\000\034\000\000\0000)\001\000\000\000\000\000\000\000\000\000\f\000\036\000\020\000\004\000\b\000\f\000\f\000\000\000\320\005\000\000$\000\000\000\030\000\000\000\300\003\000\000\000\000\000\000\000\000\000\000\000\000\006\000\b\000\a\000\006\000\000\000\000\000\000\001Z\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\000\310\t\000\000\000\000\000\000\310\t\000\000\000\000\000\000\036\000\000\000\000\000\000\000\350\t\000\000\000\000\000\000\000\000\000\000\000\000\000\000\350\t\000\000\000\000\000\000\310\t\000\000\000\000\000\000\260\023\000\000\000\000\000\000\036\000\000\000\000\000\000\000\320\023\000\000\000\000\000\000\000\000\000\000\000\000\000\000\320\023\000\000\000\000\000\000\310\t\000\000\000\000\000\000\230\035\000\000\000\000\000\000\036\000\000\000\000\000\000\000\270\035\000\000\000\000\000\000\000\000\000\000\000\000\000\000\270\035\000\000\000\000\000\000\310\t\000\000\000\000\000\000\200\'\000\000\000\000\000\000\036\000\000\000\000\000\000\000\240\'\000\000\000\000\000\000\000\000\000\000\000\000\000\000\240\'\000\000\000\000\000\000\310\t\000\000\000\000\000\000h1\000\000\000\000\000\000\036\000\000\000\000\000\000\000\2101\000\000\000\000\000\000\000\000\000\000\000\000\000\000\2101\000\000\000\000\000\000\310\t\000\000\000\000\000\000P;\000\000\000\000\000\000\036\000\000\000\000\000\000\000p;\000\000\000\000\000\000\000\000\000\000\000\000\000\000p;\000\000\000\000\000\000\310\t\000\000\000\000\000\0008E\000\000\000\000\000\000\036\000\000\000\000\000\000\000XE\000\000\000\000\000\000\000\000\000\000\000\000\000\000XE\000\000\000\000\000\000\310\t\000\000\000\000\000\000 O\000\000\000\000\000\000\036\000\000\000\000\000\000\000@O\000\000\000\000\000\000\000\000\000\000\000\000\000\000@O\000\000\000\000\000\000\310\t\000\000\000\000\000\000\bY\000\000\000\000\000\000\036\000\000\000\000\000\000\000(Y\000\000\000\000\000\000\000\000\000\000\000\000\000\000
...
...
...

We should also check other logging in the connector.

Metadata

Metadata

Assignees

Labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions