Skip to content

Temp pf - #1063

Open
mihir-datazip wants to merge 36 commits into
masterfrom
temp-pf
Open

Temp pf#1063
mihir-datazip wants to merge 36 commits into
masterfrom
temp-pf

Conversation

@mihir-datazip

Copy link
Copy Markdown
Collaborator

Description

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Scenario A
  • Scenario B

Screenshots or Recordings

Documentation

  • Documentation Link: [link to README, olake.io/docs, or olake-docs]
  • N/A (bug fix, refactor, or test changes only)

Related PR's (If Any):

@mihir-datazip
mihir-datazip temporarily deployed to Performance Testing August 3, 2026 07:33 — with GitHub Actions Inactive
@mihir-datazip
mihir-datazip temporarily deployed to Performance Testing August 3, 2026 07:33 — with GitHub Actions Inactive
@mihir-datazip
mihir-datazip temporarily deployed to Performance Testing August 3, 2026 08:30 — with GitHub Actions Inactive
@mihir-datazip
mihir-datazip temporarily deployed to Performance Testing August 3, 2026 08:30 — with GitHub Actions Inactive
@mihir-datazip
mihir-datazip temporarily deployed to Performance Testing August 4, 2026 10:49 — with GitHub Actions Inactive
@mihir-datazip
mihir-datazip temporarily deployed to Performance Testing August 4, 2026 10:49 — with GitHub Actions Inactive
mihir-datazip and others added 30 commits August 28, 2026 12:19
Co-authored-by: vikash choudhary <vikash@datazip.io>
…iceberg/olake-iceberg-java-writer (#1092)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
… /destination/iceberg/olake-iceberg-java-writer (#1147)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
…on/iceberg/olake-iceberg-java-writer (#1150)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
…1152)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
Keeps the workflow consolidation (integration-tests.yml and performance-test.yml
were replaced by tests.yml + test-preflight.yml) and carries staging's dependabot
bumps into the files that inherited those action calls:
  actions/setup-java v3/v4 -> v5
  aws-actions/configure-aws-credentials v4 -> v6
… /destination/iceberg/olake-iceberg-java-writer (#1151)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
…from 3.5.2 to 3.5.6 in /destination/iceberg/olake-iceberg-java-writer (#1156)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
…5.6.3 to 5.6.4 in /destination/iceberg/olake-iceberg-java-writer (#1157)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: vikash choudhary <vikash@datazip.io>
…on-UTC zone

--user: TestWorkingDir is a t.TempDir() bind-mounted into the container, and the
image declares no USER. On a Linux bind mount everything the driver writes there
belongs to root. A root-owned file is still removable, but a root-owned directory
is not, and pkg/indexdb creates one (olake-table-index). Go's own t.TempDir
cleanup then fails with permission denied and, reporting through t.Errorf, fails
a test whose sync had already passed.

TZ: several state-version gates only change behavior when the machine timezone is
not UTC. In a UTC container the old and new branches produce identical output, so
the compatibility suite could not tell a working gate from a removed one.

Also drops the temporary pg_class diagnostic added while chasing the concurrent
CREATE TABLE failure.
TZ=Asia/Kolkata broke the compatibility sweep in CI: the reference and upgrade
runs diverge by exactly 19800000000us (5h30m, the IST offset) on db2 col_date /
col_timestamp at state version 1 and on mysql created_timestamp at state
version 0, with no fault applied.

That divergence is worth investigating on its own -- it suggests those two gates
do not keep an old state file consistent outside UTC -- but it must not run in CI
while it does. Keeps the --user fix, which is unrelated.
Scratch-branch commit on top of backward-compatibility-tests. Never merged; drop
it when rebasing.

  - push to temp-pf triggers the workflow (it otherwise runs only on master/staging)
  - test-backward-compatibility gated exactly like integration-tests: no if:, and an
    environment expression that self-disables outside a pull request
  - performance-tests matrix populated on a push to temp-pf (was: staging only)
  - approval environments dropped from compatibility-sweep and performance-tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants