diff options
author | Reynold Xin <rxin@databricks.com> | 2015-07-19 22:45:56 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2015-07-19 22:45:56 -0700 |
commit | 5bdf16da90cf20f20e6fbb258ffd0888bf45e357 (patch) | |
tree | 8d61ad723d182b44eff642668fafba180c12675d /sql/catalyst/src | |
parent | 930253e0766a7585347edfb73ed11b1bf78143fe (diff) | |
download | spark-5bdf16da90cf20f20e6fbb258ffd0888bf45e357.tar.gz spark-5bdf16da90cf20f20e6fbb258ffd0888bf45e357.tar.bz2 spark-5bdf16da90cf20f20e6fbb258ffd0888bf45e357.zip |
Code review feedback for the previous patch.
Diffstat (limited to 'sql/catalyst/src')
-rw-r--r-- | sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala index b2468b6a18..10f411ff74 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala @@ -63,16 +63,20 @@ class CodeGenContext { * As an example, ("int", "count", "count = 0;") will produce code: * {{{ * private int count; + * }}} + * as a member variable, and add + * {{{ * count = 0; * }}} + * to the constructor. * * They will be kept as member variables in generated classes like `SpecificProjection`. */ val mutableStates: mutable.ArrayBuffer[(String, String, String)] = mutable.ArrayBuffer.empty[(String, String, String)] - def addMutableState(javaType: String, variableName: String, initialCode: String): Unit = { - mutableStates += ((javaType, variableName, initialCode)) + def addMutableState(javaType: String, variableName: String, initCode: String): Unit = { + mutableStates += ((javaType, variableName, initCode)) } final val intervalType: String = classOf[Interval].getName |