gravitino icon indicating copy to clipboard operation
gravitino copied to clipboard

[Bug report] missing assign partitions in `CapabilityHelpers`

Open xiaozcy opened this issue 1 year ago • 4 comments

Version

main branch

Describe what's wrong

Refer to #3189, I believe we are missing setting assignments in com.datastrato.gravitino.catalog.CapabilityHelpers

Error message and/or stacktrace

None

How to reproduce

None

Additional context

No response

xiaozcy avatar May 13 '24 13:05 xiaozcy

Can you describe in detail the relationship between assignments and capability?

mchades avatar May 13 '24 13:05 mchades

Can you describe in detail the relationship between assignments and capability?

In TableNormalizeDispatcher, we use applyCapabilities(partitions, capability) to wrap partition info passed by request. If we don't assign partitions in CapabilityHelpers, the table operation dispatcher would not get assignments correctly. image

xiaozcy avatar May 13 '24 13:05 xiaozcy

got it. Would you like to fix this?

mchades avatar May 13 '24 13:05 mchades

Sure, I'll fix it later.

xiaozcy avatar May 13 '24 14:05 xiaozcy