diff options
author | Reynold Xin <rxin@databricks.com> | 2016-11-01 21:20:53 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2016-11-01 21:20:53 -0700 |
commit | ad4832a9faf2c0c869bbcad9d71afe1cecbd3ec8 (patch) | |
tree | a9faa95913a019b04ec42e89ea0864b59ba53a9a /sql | |
parent | 77a98162d1ec28247053b8b3ad4af28baa950797 (diff) | |
download | spark-ad4832a9faf2c0c869bbcad9d71afe1cecbd3ec8.tar.gz spark-ad4832a9faf2c0c869bbcad9d71afe1cecbd3ec8.tar.bz2 spark-ad4832a9faf2c0c869bbcad9d71afe1cecbd3ec8.zip |
[SPARK-18216][SQL] Make Column.expr public
## What changes were proposed in this pull request?
Column.expr is private[sql], but it's an actually really useful field to have for debugging. We should open it up, similar to how we use QueryExecution.
## How was this patch tested?
N/A - this is a simple visibility change.
Author: Reynold Xin <rxin@databricks.com>
Closes #15724 from rxin/SPARK-18216.
Diffstat (limited to 'sql')
-rw-r--r-- | sql/core/src/main/scala/org/apache/spark/sql/Column.scala | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/Column.scala b/sql/core/src/main/scala/org/apache/spark/sql/Column.scala index 05e867bf5b..249408e0fb 100644 --- a/sql/core/src/main/scala/org/apache/spark/sql/Column.scala +++ b/sql/core/src/main/scala/org/apache/spark/sql/Column.scala @@ -118,6 +118,9 @@ class TypedColumn[-T, U]( * $"a" === $"b" * }}} * + * Note that the internal Catalyst expression can be accessed via "expr", but this method is for + * debugging purposes only and can change in any future Spark releases. + * * @groupname java_expr_ops Java-specific expression operators * @groupname expr_ops Expression operators * @groupname df_ops DataFrame functions @@ -126,7 +129,7 @@ class TypedColumn[-T, U]( * @since 1.3.0 */ @InterfaceStability.Stable -class Column(protected[sql] val expr: Expression) extends Logging { +class Column(val expr: Expression) extends Logging { def this(name: String) = this(name match { case "*" => UnresolvedStar(None) |