Codecov Report

Merging #2673 (50693ec) into master (322dc47) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2673      +/-   ##
==========================================
- Coverage   82.33%   82.32%   -0.01%     
==========================================
  Files         335      335              
  Lines       14047    14051       +4     
  Branches     2214     2207       -7     
==========================================
+ Hits        11565    11567       +2     
- Misses       2482     2484       +2     

It seems we have two cases where we want to retain AllColumns(*) or a single column

For example, select * from (select * from ...), we want to preserve AllColumns in the input attributes, but for just checking input columns, we need to extract an input column as in this PR.

Maybe adding an abstraction like AttributeList(inputAttributes: List[Attribute]) might be necessary to cover different use cases.

For now we have inputColumns and inputAttributes, so we can use them for different purposes.