Add explicit support for type constructors#12007
Add explicit support for type constructors#12007electrum wants to merge 1 commit intoprestodb:masterfrom electrum:constructor
Conversation
This adds an optional type constructor operator which can be used by types that have a literal form that is different than casting from varchar. Currently, the JSON type is the only such instance and support for it was previously hard-coded in several places.
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement, and yours has expired. Before we can review or merge your code, we need you to email cla@fb.com with your details so we can update your status. |
1 similar comment
|
Thank you for your pull request. We require contributors to sign our Contributor License Agreement, and yours has expired. Before we can review or merge your code, we need you to email cla@fb.com with your details so we can update your status. |
dain
left a comment
There was a problem hiding this comment.
I no longer work on prestodb, and instead am focused full time on https://github.com/prestosql/presto
|
This pull request has been automatically marked as stale because it has not had recent activity. If you'd still like this PR merged, please comment on the task, make sure you've addressed reviewer comments, and rebase on the latest master. Thank you for your contributions! |
This adds an optional type constructor operator which can be used by
types that have a literal form that is different than casting from
varchar. Currently, the JSON type is the only such instance and
support for it was previously hard-coded in several places.