Add ADR for column visibility feature#2222
Conversation
Co-authored-by: jzabroski <[email protected]>
|
@copilot You missed some providers. DB2 supports it via "IMPLICITLY HIDDEN" syntax; See https://www.ibm.com/docs/en/db2/12.1.x?topic=concepts-hidden-columns
|
Co-authored-by: jzabroski <[email protected]>
Added DB2 support with |
|
@copilot
|
Co-authored-by: jzabroski <[email protected]>
Restructured the table with separate columns for CREATE TABLE, ALTER TABLE ADD COLUMN, and ALTER TABLE ALTER COLUMN syntax. Updated DB2 to include both |
|
@PhenX Think this looks great. Thoughts? I agree with copilot's default recommendation |
|
Yep looks good! |
|
I think my new approach is going to be to model feature enhancements this way. First have Copilot explain its thinking, then create a separate issue for it to go implement it. |
Many databases support hiding columns from
SELECT *queries (SQL Server'sHIDDEN, Oracle/MySQL/MariaDB'sINVISIBLE, DB2'sIMPLICITLY HIDDEN). FluentMigrator currently requires raw SQL for this feature. This ADR documents database support and proposes a fluent API design.Changes
adr/proposed/ColumnVisibility.mddocumenting:Hidden()andVisible()methods on column buildersProposed API
No implementation included per issue requirements.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.