Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7348#discussion_r34522122
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -335,15 +347,31 @@ abstract class BinaryExpression extends Expression
with trees.BinaryNode[Express
/**
- * An expression that has two inputs that are expected to the be same
type. If the two inputs have
- * different types, the analyzer will find the tightest common type and do
the proper type casting.
+ * A [[BinaryExpression]] that is an operator, with two properties:
+ *
+ * 1. The string representation is "x symbol y", rather than "funcName(x,
y)".
+ * 2. Two inputs are expected to the be same type. If the two inputs have
different types,
+ * the analyzer will find the tightest common type and do the proper
type casting.
*/
abstract class BinaryOperator extends BinaryExpression {
self: Product =>
def symbol: String
--- End diff --
that's a mistake actually. i'm adding tostirng back
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
