Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge 'Fix multi-cell static list updates in the presence of ckeys' f…
…rom Duarte Fixes a regression introduced in 9e88b60, which broke the lookup for prefetched values of lists when a clustering key is specified. This is the code that was removed from some list operations: std::experimental::optional<clustering_key> row_key; if (!column.is_static()) { row_key = clustering_key::from_clustering_prefix(*params._schema, prefix); } ... auto&& existing_list = params.get_prefetched_list(m.key().view(), row_key, column); Put it back, in the form of common code in the update_parameters class. Fixes #3703 * https://github.com/duarten/scylla cql-list-fixes/v1: tests/cql_query_test: Test multi-cell static list updates with ckeys cql3/lists: Fix multi-cell static list updates in the presence of ckeys keys: Add factory for an empty clustering_key_prefix_view (cherry picked from commit 6937cc2)
- Loading branch information
Showing
3 changed files
with
70 additions
and
0 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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