I fixed the checker framework problems. One of the last ones - continuous integration - ran out of memory. Is that flakiness or something I need to fix?
Thanks, -Paul On Monday, January 8, 2024 at 05:41:30 PM EST, Julian Hyde <[email protected]> wrote: I see two failures due to checkerframework. Those are real. Let us know when you've fixed them. The failure in Druid is probably due to a Druid bug that existed in November. You can ignore it. Should go away after rebase. Please keep nagging on this thread... this is a good change, and we should get it into 1.37. On Mon, Jan 8, 2024 at 11:34 AM Mihai Budiu <[email protected]> wrote: > > CI failed, so perhaps you can look into that. > > Mihai > ________________________________ > From: Paul Jackson <[email protected]> > Sent: Monday, January 8, 2024 7:06 AM > To: [email protected] <[email protected]> > Subject: Re: PR 3495 (Make single-row rels unique) Review Request > > Is there anything else to do for this PR? > > * https://github.com/apache/calcite/pull/3495 > * https://issues.apache.org/jira/browse/CALCITE-6044 > > Thanks, > -Paul > > > > On Nov 16, 2023, at 7:51 AM, Paul Jackson > > <[email protected]> wrote: > > > > This PR updates org.apache.calcite.rel.metadata areColumnsUnique, > > getUniqueKeys, and getPredicates methods to account for relations that > > produce at most a single row: Sort (due to limit), Aggregate (due to no > > group by), Values (due to having a single row), and Join (when joining with > > one of the other nodes). > > https://github.com/apache/calcite/pull/3495https://issues.apache.org/jira/projects/CALCITE/issues/CALCITE-6044 > > Thanks,-Paul
