Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

airframe-sql: Intersect should have attributes of all source relations #2589

Merged
merged 3 commits into from Nov 25, 2022

Conversation

takezoe
Copy link
Member

@takezoe takezoe commented Nov 25, 2022

In the same way as Union

@codecov
Copy link

codecov bot commented Nov 25, 2022

Codecov Report

Merging #2589 (894d390) into master (fda8293) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2589      +/-   ##
==========================================
+ Coverage   81.97%   82.01%   +0.04%     
==========================================
  Files         333      333              
  Lines       13734    13748      +14     
  Branches     2117     2148      +31     
==========================================
+ Hits        11259    11276      +17     
+ Misses       2475     2472       -3     
Impacted Files Coverage Δ
...scala/wvlet/airframe/sql/parser/SQLGenerator.scala 89.08% <ø> (+0.03%) ⬆️
...ala/wvlet/airframe/sql/analyzer/TypeResolver.scala 94.33% <100.00%> (+0.40%) ⬆️
...n/scala/wvlet/airframe/sql/model/LogicalPlan.scala 86.66% <100.00%> (+1.27%) ⬆️
...ala/wvlet/airframe/sql/parser/SQLInterpreter.scala 87.18% <100.00%> (+0.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fda8293...894d390. Read the comment docs.

@xerial
Copy link
Member

xerial commented Nov 25, 2022

Build is failing for Scala 3 🤔

@xerial
Copy link
Member

xerial commented Nov 25, 2022

Removed debug logs

@takezoe
Copy link
Member Author

takezoe commented Nov 25, 2022

Ah, remaining debug code was not Scala3 compatible. 🙇‍♂️

case class Intersect(relations: Seq[Relation], nodeLocation: Option[NodeLocation]) extends SetOperation {
case class Intersect(
relations: Seq[Relation],
resolvedOutputs: Option[Seq[Attribute]],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@xerial xerial merged commit 2530f62 into wvlet:master Nov 25, 2022
@xerial xerial added the bug label Nov 25, 2022
@takezoe takezoe deleted the sql-resolve-intersect branch December 5, 2022 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants