Skip to content

Conversation

xiaozcy
Copy link
Contributor

@xiaozcy xiaozcy commented Jun 11, 2025

What changes were proposed in this pull request?

Fix get file location with multiple locations.

Why are the changes needed?

Fix: #7371

Does this PR introduce any user-facing change?

No.

How was this patch tested?

UT.

Copy link
Contributor

@mchades mchades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@mchades mchades added the branch-0.9 Automatically cherry-pick commit to branch-0.9 label Jun 11, 2025
@mchades mchades merged commit 89b8b82 into apache:main Jun 11, 2025
30 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 11, 2025
…7372)

### What changes were proposed in this pull request?

Fix get file location with multiple locations.

### Why are the changes needed?

Fix: #7371

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

UT.

Co-authored-by: zhanghan <zhanghan18@xiaomi.com>
yunchipang pushed a commit to yunchipang/gravitino that referenced this pull request Jun 11, 2025
…ions (apache#7372)

### What changes were proposed in this pull request?

Fix get file location with multiple locations.

### Why are the changes needed?

Fix: apache#7371

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

UT.

Co-authored-by: zhanghan <zhanghan18@xiaomi.com>
jerryshao pushed a commit to jerryshao/gravitino that referenced this pull request Jul 8, 2025
…ions (apache#7372)

### What changes were proposed in this pull request?

Fix get file location with multiple locations.

### Why are the changes needed?

Fix: apache#7371

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

UT.

Co-authored-by: zhanghan <zhanghan18@xiaomi.com>
vishnu-chalil pushed a commit to vishnu-chalil/gravitino that referenced this pull request Jul 14, 2025
…ions (apache#7372)

### What changes were proposed in this pull request?

Fix get file location with multiple locations.

### Why are the changes needed?

Fix: apache#7371

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

UT.

Co-authored-by: zhanghan <zhanghan18@xiaomi.com>
hdygxsj pushed a commit to hdygxsj/gravitino that referenced this pull request Jul 15, 2025
…ions (apache#7372)

### What changes were proposed in this pull request?

Fix get file location with multiple locations.

### Why are the changes needed?

Fix: apache#7371

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

UT.

Co-authored-by: zhanghan <zhanghan18@xiaomi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-0.9 Automatically cherry-pick commit to branch-0.9
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug report] Fileset location is incorrect when getting path context from fileset with multiple locations
2 participants