avantgardnerio opened a new pull request, #5835: URL: https://github.com/apache/arrow-datafusion/pull/5835
# Which issue does this PR close? Closes #5834. # Rationale for this change As a developer building a database atop DataFusion, it would be useful to have knowledge of primary keys retained in the LogicalPlan. # What changes are included in this PR? A new `primary_key` field on `LogicalPlan::CreateMemoryTable` # Are these changes tested? Sort of - there's a test that broke before, but I couldn't figure out what prints this LogicalPlan node, so I wasn't able to add the pk info to the `Debug` trait and assert that. # Are there any user-facing changes? `create table` statements that used to fail (e.g. with `primary key` clauses) will now not fail. I'm not sure if that's an issue for anyone. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
