keepConcentration commented on PR #8695: URL: https://github.com/apache/gravitino/pull/8695#issuecomment-3395648517
> > > > > @keepConcentration It's expected that CI will fail after your change. Do you happen to have time to try to fix the failure? Just to let you know, this is not a major issue. > > > > > > > > > > > > Hi, @yuqi1129 I ran the tests locally and they all pass. Could you share which specific tests are expected to fail in CI? Understanding the failure cases would help me address the root cause. > > > > Thank you. > > > > > > > > > Please closely watch the failure test cases in the CI. > > > > > > It seems that intentionally not caching relational data is a design choice. Based on PR [#8297](https://github.com/apache/gravitino/pull/8297), specifically the commit [fba3b80](https://github.com/apache/gravitino/commit/fba3b80be1df177c2262e2210f6927b416ce6e91) (Disable cache multi-layer relation entity), this behavior was deliberately added. > > I think so, and we can close this PR if it's hard to move on. Thanks for confirming. -- 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]
