-
-
Notifications
You must be signed in to change notification settings - Fork 340
1.8 #910
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* calls template<class... Tss, class... Cols>
sync_schema_result schema_status(const storage_impl<index_t<Cols...>, Tss...>&, sqlite3*, bool) {
return sync_schema_result::already_in_sync;
}
* Initial enhancement to sync_schema() internals so as to not lose data
* adding changes to index.cpp
* Replaced table_info with table_xinfo all over the library and corrected the treatment of generated columns
* amalgamate sources and clang-formatted all
* Added table_data_loss to enum sync_schema_result, modified sync_schema_tests.cpp to use table_data_loss and all tests are passing
* support for table_data_loss in enum sync_schema_result and all files formatted with clang-format
* commit to push according to Klaus
* refactoring: moved all methods outside of storage_impl_base
* Added drop_create_with_loss() to handle sync_schema_result::table_data_loss: into a migration framed function
* Fixed all comments from my last PR
* corrected Migration API frame so that exceptions in rollback or commit do not leave FK OFF
* made foreign_keys(bool) noexcept to make exceptions from setting FK checking flag a program termination
* clang-format for FK setting as a noexcept function
* fixed foreign keys so they will be taken care when opening connection and not erasing FK setting with true in on_open_terminal
* Support for generated stored columns where the update method was not working
* Added final_action class and conditionally compiled out any FK checking OFF!!
* Perfecting conditional compilation support for NO FK checking OFF
* Removed comments that were valid when toggling FK enabled
* Removing FK toggling - step 1
* removing table_data_loss step 1
* removing table_data_loss - step 2
* Implemented case for new column with no default value and notnull and removed code for FK toggling according to my conversation with Eugene
* Removed new value for sync_schema_result and created a parameter for communication between schema_status() and sync_table() to avoid trying to preserve values when this is impossible: the new column with no defaults and not nullable case
* Removed #if 0 //jdh in pragma.h
* Ran amalgamation and clang-format
* Corrected all comments by Eugene for acceptance of PR
* Fixed similar rename_tables in storage_base.h so one delegates to the other...
* Fixing every comment made by Eugene except about transactions
* amalgamete and clang-format all files
* removed transactions inside library and all //jdh comments
* resolved all issues commented by Eugene - should be ready to merge
* moved implementation of calculate_remove_add_columns to storage_base.h
was getting multiple definitions from the linker
This change was necessary because of suspected possible ODR violation for msvc 14.0, which made unit tests crash or behave strangely. Only those in tests/operators/glob.cpp, tests/select_constraints_tests.cpp, tests/unique_cases/issue663.cpp were affected at this commit
… in an example instead of a test
Implemented sys_days as bindable type; a C++ 20 feature
|
It looks like a couple of the CI checks hung? |
|
@NickStrupat yeah. Let me restart it |
|
@NickStrupat IDK how to run it explicitly. Actually it is not a problem cause I build check on AppVeyor is enough. R u waiting for 1.8? |
added column_name function to prepared_statement_base
Corrected storage lookup in regard to non-unique DB objects
added on conflict for primary key
refactored legacy autoincrement usage
removed has_dependent_rows
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.