mxnzx commented on issue #8084: URL: https://github.com/apache/gravitino/issues/8084#issuecomment-3188346469
Hi, I added a property and my unit tests pass locally. However, when I run the whole unit test suite with: `./gradlew test -PskipITs` I get a failure in TestMetadataObjectTagOperations.testListTagsForObject(): ``` expected: <4> but was: <6> at org.apache.gravitino.server.web.rest.TestMetadataObjectTagOperations.testListTagsForObject(TestMetadataObjectTagOperations.java:367) ``` The docs seem to suggest that if the same tag is attached to both a parent and a child, the child’s tag list may include two entries with different inherited values. But this test appears to expect only one entry per tag name, effectively preferring the direct tag over the inherited one, resulting in a total of 4. Could you clarify the intended behavior for this endpoint? Should the API return both entries (direct and inherited) when names collide (leading to 6 in this case)? Or is the intended spec to deduplicate by tag name and prefer direct > inherited, keeping the count at 4? Depending on your guidance, I can either adjust the server logic (dedup by name with direct precedence) or update the test expectation/documentation accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
