-
Notifications
You must be signed in to change notification settings - Fork 93
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
MergeIntoNodeBuilder: java.lang.IllegalArgumentException: transpose #605
Comments
wajda
added a commit
that referenced
this issue
Feb 16, 2023
wajda
added a commit
that referenced
this issue
Feb 16, 2023
wajda
added a commit
that referenced
this issue
Feb 16, 2023
wajda
added a commit
that referenced
this issue
Feb 28, 2023
wajda
added a commit
that referenced
this issue
Feb 28, 2023
wajda
added a commit
that referenced
this issue
Feb 28, 2023
…neageHarvester` to `OperationNodeBuilderFactory`, and delegate `MergeIntoCommand` children extraction logic to the `MergeIntoNodeBuilder` respectively.
wajda
added a commit
that referenced
this issue
Feb 28, 2023
…orythm to accommodate for renaming and sub-attributes in the MERGE clauses.
wajda
added a commit
that referenced
this issue
Mar 3, 2023
… as agreed on PR discussion. Use the type alias in all places where appropriate.
wajda
added a commit
that referenced
this issue
Mar 3, 2023
…606) * spline #605 MergeIntoNodeBuilder: java.lang.IllegalArgumentException: transpose * spline #605 Addressing PR comment: rename `src` (that is of type Option) to `maybeSrc` * spline #605 minor code style: remove redundant `override val` * spline #605 minor Refactoring and code style * spline #605 Refactoring: move `extractChildren(Node)` method from `LineageHarvester` to `OperationNodeBuilderFactory`, and delegate `MergeIntoCommand` children extraction logic to the `MergeIntoNodeBuilder` respectively. * spline #605 Reimplement MERGE INTO attribute dependency resolving algorythm to accommodate for renaming and sub-attributes in the MERGE clauses. * spline #605 Refactoring: Rename type alias Attributes -> IOAttributes as agreed on PR discussion. Use the type alias in all places where appropriate.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Originally posted by @wajda in #570 (comment)
The text was updated successfully, but these errors were encountered: