apache / apache/polaris

Odd `/` and `//` paths in JDBC optimized location overlap checks

Closed
#5,023 1 comment 0 reactions 0 assignees View on GitHub
stale
Dominant language
Java
Stars
2.1k
Forks
522
Avg merge
2d 16m
Merged PRs (30d)
139

Description

Based on: https://github.com/apache/polaris/pull/5003#discussion_r3552655603

`persistence/relational-jdbc/.../QueryGenerator` uses paths like `/` and `//` in location overlap checks on S3 locations (as an example)

These paths do not look correct when they are derived from the URI host separator, e.g. `s3://bucket/path`

Contributor guide

Open the contributing guide

Research direction

Start by reading persistence/relational-jdbc/.../QueryGenerator, focusing on the location overlap checks and how URI host separators produce paths for locations such as s3://bucket/path. Trace the generated values for / and //, then verify that the overlap checks use the intended location paths without extra separators.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
database
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.