Skip to content

feat: Add daily time partitioning to BigQuery event logging table - #1377

Draft
vasilyl wants to merge 1 commit into
google:mainfrom
vasilyl:table-partitioning
Draft

feat: Add daily time partitioning to BigQuery event logging table#1377
vasilyl wants to merge 1 commit into
google:mainfrom
vasilyl:table-partitioning

Conversation

@vasilyl

@vasilyl vasilyl commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Please ensure you have read the contribution guide before creating a pull request.

Link to Issue or Description of Change

1. Link to an existing issue (if applicable):

2. Or, if no issue exists, describe the change:

Problem:
Logging large volumes of events into a flat BigQuery table can lead to inefficient query scanning, resulting in slower analytical queries and higher query costs over time.

Solution:
Configure daily time partitioning during table creation inside NewBigQueryAgentAnalyticsPluginWithClients. The BigQuery table is created with TimePartitioning enabled on the timestamp field using bq.DayPartitioningType.

Testing Plan

Added unit test TestNewBigQueryAgentAnalyticsPlugin_CreateTable_WithPartitioning to verify that table creation requests correctly specify the partitioning parameters (timePartitioning with type DAY and field timestamp).

Checklist

  • I have read the CONTRIBUTING.md document.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have manually tested my changes end-to-end.

@vasilyl
vasilyl force-pushed the table-partitioning branch from 3b532e0 to 8615e48 Compare August 21, 2026 23:29
@vasilyl
vasilyl force-pushed the table-partitioning branch from 8615e48 to b46b96d Compare August 25, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant