Re: [PATCH] Keeps tracking the uniqueness with UniqueKey

Hi,

On 11/30/20 5:04 AM, Heikki Linnakangas wrote:
> On 26/11/2020 16:58, Andy Fan wrote:
>> This patch has stopped moving for a while,  any suggestion about
>> how to move on is appreciated.
>
> The question on whether UniqueKey.exprs should be a list of 
> EquivalenceClasses or PathKeys is unresolved. I don't have an opinion 
> on that, but I'd suggest that you pick one or the other and just go 
> with it. If it turns out to be a bad choice, then we'll change it.

In this case I think it is matter of deciding if we are going to use 
EquivalenceClasses or Exprs before going further; there has been work 
ongoing in this area for a while, so having a clear direction from a 
committer would be greatly appreciated.

Deciding would also help potential reviewers to give more feedback on 
the features implemented on top of the base.

Should there be a new thread with the minimum requirements in order to 
get closer ?

Best regards,
  Jesper