You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "jonahgao (via GitHub)" <gi...@apache.org> on 2023/11/07 15:56:38 UTC
[PR] feat: support target table alias in update statement [arrow-datafusion]
jonahgao opened a new pull request, #8080:
URL: https://github.com/apache/arrow-datafusion/pull/8080
## Which issue does this PR close?
N/A.
Support statements like `UPDATE foo AS f `.
## Rationale for this change
> When an alias is provided, it completely hides the actual name of the table. For example, given UPDATE foo AS f, the remainder of the UPDATE statement must refer to this table as f not foo.
Ref: [PostgreSQL UPDATE SQL Commands](https://www.postgresql.org/docs/current/sql-update.html)
## What changes are included in this PR?
If there is an alias for the target table, use it to properly qualify the column names from the target table.
## Are these changes tested?
Yes
## Are there any user-facing changes?
No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat: support target table alias in update statement [arrow-datafusion]
Posted by "jonahgao (via GitHub)" <gi...@apache.org>.
jonahgao commented on PR #8080:
URL: https://github.com/apache/arrow-datafusion/pull/8080#issuecomment-1804956281
> Looks good to me @jonahgao -- thank you for the contribution (again). These improvements are really appreciated.
>
> You might be interested in #8008 which is also related to qualified names.
Thank you for your review @alamb .
I will pay attention to this issue and hope to be helpful.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat: support target table alias in update statement [arrow-datafusion]
Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb merged PR #8080:
URL: https://github.com/apache/arrow-datafusion/pull/8080
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org