Thanks for sharing, Dmitri! The change is positive. I'm wondering if we could even simplify further by not allowing two expire-at properties in the connectiion credentials.
Yufei On Wed, Apr 15, 2026 at 2:18 PM Dmitri Bourlatchkov <[email protected]> wrote: > Hi All, > > PR [4173] adds extra validation to ConnectionCredentials to avoid ambiguity > WRT expiration timestamps. > > I believe this is a solid fix worth merging. > > However, there is a potential for breakage in environments that previously > had incorrect configuration. > > All in all, the PR has been in review for 3 days. I propose merging > tomorrow. > > [4173] https://github.com/apache/polaris/pull/4173 > > Cheers, > Dmitri. >
