Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unspecified table dolt diff and dolt_history should show the history of all the tables per commit #2740

Closed
timsehn opened this issue Feb 2, 2022 · 0 comments · Fixed by #2769
Assignees

Comments

@timsehn
Copy link
Contributor

timsehn commented Feb 2, 2022

User wanted something akin to git log -p --name-only but in SQL.

Seems like an unscoped dolt_history table would be useful.

something like:

select * from dolt_history;

table, commit_hash, committer, commit_date
table1, abcd, tim, 'feb 2, 2022'
table2, abcd, tim, 'feb 2, 2022'
table1,wxyx, tim, 'feb 1, 2022'

I'm not sure this gets exactly what he needs. This would allow you to see tables added and removed.

Maybe unscoped dolt diff? From the above, it would look the same but would imply each table changed in both those commits. Imagine there was a table 0 that did not change.

select * from dolt_history;

table, commit_hash, committer, commit_date
table0,  abcd, tim, 'feb 2, 2022'
table1, abcd, tim, 'feb 2, 2022'
table2, abcd, tim, 'feb 2, 2022'
table1,wxyx, tim, 'feb 1, 2022'
table0, wxyx, tim, 'feb 1, 2022' 

select * from dolt_diff

table, commit_hash, committer, commit_date
table1, abcd, tim, 'feb 2, 2022'
table2, abcd, tim, 'feb 2, 2022'
table1,wxyx, tim, 'feb 1, 2022'
table0, wxyx, tim, 'feb 1, 2022' 
@timsehn timsehn changed the title Unspecified table dolt_history should show the history of all the tables per commit Unspecified table dolt diff and dolt_history should show the history of all the tables per commit Feb 2, 2022
@fulghum fulghum self-assigned this Feb 4, 2022
@fulghum fulghum added the feature label Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants