Skip to content

Antalya 26.3 Backport - Added test cases for s3 encoding fix#1781

Open
subkanthi wants to merge 2 commits into
antalya-26.3from
1338_fix_3_decoding_antaly_26_3
Open

Antalya 26.3 Backport - Added test cases for s3 encoding fix#1781
subkanthi wants to merge 2 commits into
antalya-26.3from
1338_fix_3_decoding_antaly_26_3

Conversation

@subkanthi
Copy link
Copy Markdown
Collaborator

@subkanthi subkanthi commented May 11, 2026

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Backport of #1516, unit/integration tests added. The original fix was in the resolveObjectStorageForPath function(Utils.cpp) which was renamed/rewritten to getProperFilePathFromMetadataInfo. The tests are to make sure % encoded URLs still work in iceberg. closes #1348

Documentation entry for user-facing changes

Backport of #1516, unit tests added.

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Performance tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All with Aarch64
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 11, 2026

Workflow [PR], commit [e2623e2]

@subkanthi subkanthi changed the title Added test cases for s3 encoding fix Antalya 26.3 Backport - Added test cases for s3 encoding fix May 13, 2026
@subkanthi subkanthi marked this pull request as ready for review May 13, 2026 19:55
@subkanthi
Copy link
Copy Markdown
Collaborator Author

tests/integration/test_database_iceberg/test.py::test_partition_value_with_slash PASSED                            [ 75%]
Note: Google Test filter = GetProperFilePathFromMetadataInfo.*
[==========] Running 4 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 4 tests from GetProperFilePathFromMetadataInfo
[ RUN      ] GetProperFilePathFromMetadataInfo.S3SchemePreservesPercentEncodedSlash
[       OK ] GetProperFilePathFromMetadataInfo.S3SchemePreservesPercentEncodedSlash (0 ms)
[ RUN      ] GetProperFilePathFromMetadataInfo.SimpleKeyWithoutEncoding
[       OK ] GetProperFilePathFromMetadataInfo.SimpleKeyWithoutEncoding (0 ms)
[ RUN      ] GetProperFilePathFromMetadataInfo.MultiplePercentEncodedSegments
[       OK ] GetProperFilePathFromMetadataInfo.MultiplePercentEncodedSegments (0 ms)
[ RUN      ] GetProperFilePathFromMetadataInfo.HttpSchemePreservesPercentEncodedSlash
[       OK ] GetProperFilePathFromMetadataInfo.HttpSchemePreservesPercentEncodedSlash (0 ms)
[----------] 4 tests from GetProperFilePathFromMetadataInfo (0 ms total)

[----------] Global test environment tear-down
[==========] 4 tests from 1 test suite ran. (0 ms total)
[  PASSED  ] 4 tests.

@svb-alt svb-alt added port-antalya PRs to be ported to all new Antalya releases forwardport This is a frontport of code that existed in previous Antalya versions labels May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forwardport This is a frontport of code that existed in previous Antalya versions port-antalya PRs to be ported to all new Antalya releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants