[SQL] Support transforming TreeNodes with Option children.

[concretevitamin]

Thanks goes to @marmbrus for his implementation.

[AmplabJenkins]

Merged build triggered.

[AmplabJenkins]

Merged build started.

[AmplabJenkins]

Merged build finished.

[AmplabJenkins]

Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15752/

[marmbrus]

This test is failing because you made the TreeNode an inner class. An inner class is actually just syntactic sugar done by the compiler where there is an extra implicit constructor parameter to the class for the outer object. As a result, the contract that all of the argument to the TreeNode's constructor are either in productIterator or otherCopyArgs is broken.

Here's a fix: concretevitamin#1

[concretevitamin]

Ah, this is tricky. Good to know and thanks for the fix.

On Friday, June 13, 2014, Michael Armbrust [email protected] wrote:

This test is failing because you made the TreeNode an inner class. An
inner class is actually just syntactic sugar done by the compiler where
there is an extra implicit constructor parameter to the class for the outer
object. As a result, the contract that all of the argument to the TreeNode's
constructor are either in productIterator or otherCopyArgs is broken.

Here's a fix: concretevitamin#1
concretevitamin#1

—
Reply to this email directly or view it on GitHub
#1074 (comment).

[AmplabJenkins]

Merged build triggered.

[AmplabJenkins]

Merged build started.

[AmplabJenkins]

Merged build finished. All automated tests passed.

[AmplabJenkins]

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15779/