Skip to content

[SPARK-22856][SQL] Add wrappers for codegen output and nullability #20043

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.

Sign up for GitHub

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

Closed
viirya wants to merge 14 commits into from
Closed

[SPARK-22856][SQL] Add wrappers for codegen output and nullability #20043

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
5ace8b8
Add wrappers for codegen output.
viirya Dec 21, 2017
d120750
Merge remote-tracking branch 'upstream/master' into SPARK-22856
viirya Dec 21, 2017
81c9b6e
Fix equality check with string.
viirya Dec 21, 2017
53926cc
Address comments.
viirya Dec 21, 2017
69422d4
Merge remote-tracking branch 'upstream/master' into SPARK-22856
viirya Dec 22, 2017
4384c84
Fix merging.
viirya Dec 22, 2017
d864aff
Merge remote-tracking branch 'upstream/master' into SPARK-22856
viirya Feb 21, 2018
8715d32
Add test.
viirya Feb 21, 2018
f59bb19
Use TrueLiteral/FalseLiteral. Add java type and access property to Ex…
viirya Feb 28, 2018
37ae9b0
Address comment.
viirya Mar 1, 2018
0841c4a
Address comment.
viirya Mar 1, 2018
e530f01
Remove ExprType.
viirya Mar 1, 2018
c8c70a9
Merge remote-tracking branch 'upstream/master' into SPARK-22856
viirya Mar 5, 2018
ac2e595
Merge remote-tracking branch 'upstream/master' into SPARK-22856
viirya Apr 4, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'upstream/master' into SPARK-22856
  • Loading branch information
@viirya
viirya committed Dec 21, 2017
commit d120750ff61bb066e7ceb628f3356fa37af462f5
10 changes: 5 additions & 5 deletions sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -214,11 +214,11 @@ case class Stack(children: Seq[Expression]) extends Generator {

// Create the collection.
val wrapperClass = classOf[mutable.WrappedArray[_]].getName
ctx.addMutableState(
s"$wrapperClass<InternalRow>",
ev.value,
v => s"$v = $wrapperClass$$.MODULE$$.make($rowData);", useFreshName = false)
ev.copy(code = code, isNull = LiteralValue("false"))
ev.copy(code =
s"""
|$code
|$wrapperClass<InternalRow> ${ev.value} = $wrapperClass$$.MODULE$$.make($rowData);
""".stripMargin, isNull = LiteralValue("false"))
}
}

Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.