Skip to content

[SPARK-29837][SQL] PostgreSQL dialect: cast to boolean #26463

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
Ngone51 wants to merge 17 commits into from
Closed

[SPARK-29837][SQL] PostgreSQL dialect: cast to boolean #26463

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
75d51ba
postgresql dialect: cast to boolean
Ngone51 Nov 11, 2019
53c9a9b
fix PostgreSQLDialectQuerySuite
Ngone51 Nov 11, 2019
2ab1847
fix postgreSQL/boolean.sql
Ngone51 Nov 11, 2019
6c2c0c7
nullable should be false now
Ngone51 Nov 12, 2019
6ad100c
check unsupported types in checkInputDataTypes()
Ngone51 Nov 12, 2019
101a026
private[catalyst] -> protected[this]
Ngone51 Nov 12, 2019
be08daf
remove unused import
Ngone51 Nov 12, 2019
95caf40
return TypeCheckSuccess
Ngone51 Nov 12, 2019
8ebd5aa
use collect()
Ngone51 Nov 12, 2019
0b9a78d
use child.nullable
Ngone51 Nov 12, 2019
fda48c0
excludes unsupported types explicitly
Ngone51 Nov 12, 2019
36f60ee
protected[this] -> protected
Ngone51 Nov 12, 2019
b7c6baf
throw exception for PostgreCastToBoolean.ansiEnabled
Ngone51 Nov 12, 2019
b9711ae
include supportted types explicitly
Ngone51 Nov 13, 2019
68c7a13
adjust tests
Ngone51 Nov 13, 2019
999ec61
missing BooleanType
Ngone51 Nov 13, 2019
a178b79
leave cast( bool as bool) to optimizer
Ngone51 Nov 13, 2019
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
private[catalyst] -> protected[this]
  • Loading branch information
@Ngone51
Ngone51 committed Nov 12, 2019
commit 101a0267385ce1de4e75aaa29c87013facade28a
2 changes: 1 addition & 1 deletion sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@ abstract class CastBase extends UnaryExpression with TimeZoneAwareExpression wit
private[this] def needsTimeZone: Boolean = Cast.needsTimeZone(child.dataType, dataType)

// [[func]] assumes the input is no longer null because eval already does the null check.
@inline private[catalyst] def buildCast[T](a: Any, func: T => Any): Any = func(a.asInstanceOf[T])
@inline protected[this] def buildCast[T](a: Any, func: T => Any): Any = func(a.asInstanceOf[T])
Copy link
Contributor

@cloud-fan cloud-fan Nov 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I don't think [this] here can buy us anything. How about just use protected?


private lazy val dateFormatter = DateFormatter(zoneId)
private lazy val timestampFormatter = TimestampFormatter.getFractionFormatter(zoneId)
Expand Down