You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "Jefffrey (via GitHub)" <gi...@apache.org> on 2023/02/05 00:29:58 UTC

[GitHub] [arrow-datafusion] Jefffrey commented on pull request #5183: Parse identifiers properly for TableReferences

Jefffrey commented on PR #5183:
URL: https://github.com/apache/arrow-datafusion/pull/5183#issuecomment-1416885998

   @alamb you mentioned possibility of removing `OwnedTableReference` https://github.com/apache/arrow-datafusion/issues/4532#issuecomment-1415727576
   
   however even though using `Cow` now for `TableReference` so could own the string, it still carries lifetime requirements which would make it tedious to replace usages of `OwnedTableReference` with `TableReference` and having to ensure all the lifetime requirements, so will keep it around for now (unless theres some other solution)


-- 
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