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

Posted to github@arrow.apache.org by "ywc88 (via GitHub)" <gi...@apache.org> on 2023/10/02 22:34:07 UTC

Re: [PR] feat(c/driver/postgresql,c/driver/sqlite): Implement FOREIGN KEY constraints [arrow-adbc]

ywc88 commented on PR #1099:
URL: https://github.com/apache/arrow-adbc/pull/1099#issuecomment-1743863928

   Sorry @OleMussmann for the delayed review, I was out sick most of last week and didn't get a chance to look at this until today.  Other than a minor nit which I left a comment on, I appreciate how comprehensive your tests are and don't see anything that could be an issue (e.g. resource leak) after checking this PR out locally and playing with it.  @lidavidm could you take a second look just in case I missed something?


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