{"push_id":3143142638,"size":16,"distinct_size":16,"ref":"refs/heads/next","head":"9fa60f63ae38b5cc5f892c24b8fa470b901597d9","before":"571755e117c33882478f16a06d4eb1ddd1c7cd41","commits":[{"sha":"2ddb99620b34e4bcfcc31cd03938cbcc5dbadbba","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"reader_concurrency_semaphore: add counters for inactive reads\n\nAdd counters that give insight into inactive read related events.\nTwo counters are added:\n* permit_based_evictions\n* population","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/2ddb99620b34e4bcfcc31cd03938cbcc5dbadbba"},{"sha":"125bccabaf51db097b6b343caf1a78614c48ad7b","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"database: s/active_reads*/ongoing_reads*/\n\nNow that we have the `inactive_reads` counters for inactive reads, the\n`active_reads*` counters are misleading because `inactive_reads` is\nactually a subset of `active_reads`, whereas one would expect these sets\nto be mutually exclusive based on their namse.\nTo avoid the confusion, rename active reads to ongoing reads. This term\nalso better describes what these reads are: they are reads that are\ncurrently ongoing, but not necessarily active.","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/125bccabaf51db097b6b343caf1a78614c48ad7b"},{"sha":"5ad01c129567b1ce19fae7f554b99b6c14ec081c","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"treewide: prepare for a move-only inactive read handle","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/5ad01c129567b1ce19fae7f554b99b6c14ec081c"},{"sha":"9a8a059973076ae6a4b73aef818314ade429f46d","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"reader_concurrency_semaphore: make the inactive read handle a RAII object","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/9a8a059973076ae6a4b73aef818314ade429f46d"},{"sha":"e896161fe3de5efae413dc6b2d8944ca413467d7","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"reader_concurrency_semaphore: make register_inactive_read a template","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/e896161fe3de5efae413dc6b2d8944ca413467d7"},{"sha":"f5b18f8400fd5ea75479dc55d41cecce5dde160a","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"tests/querier_cache: prepare for counter migration","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/f5b18f8400fd5ea75479dc55d41cecce5dde160a"},{"sha":"4b2f3d15fa1d314326a4afa504d7e907b7d1b0e5","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"querier_cache: delegate resource based eviction stats to r_c_s","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/4b2f3d15fa1d314326a4afa504d7e907b7d1b0e5"},{"sha":"12acbae97924c4502350a69b662f550d13c3183e","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"reader_concurrency_semaphore: add memory limit","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/12acbae97924c4502350a69b662f550d13c3183e"},{"sha":"11a252c9161af07b513cdd934874e6d19c93b1fc","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"querier: delegate memory limit enforcement to reader_concurrency_semaphore","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/11a252c9161af07b513cdd934874e6d19c93b1fc"},{"sha":"8ac6a350ac61f55f3dcdd6135784906a6f2f43dc","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"reader_concurrency_semaphore: add support for expiring inactive reads","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/8ac6a350ac61f55f3dcdd6135784906a6f2f43dc"},{"sha":"d7587084f7227d9d6381021d5a683d5938e05397","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"querier: delegate expiry to reader_concurrency_semaphore","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/d7587084f7227d9d6381021d5a683d5938e05397"},{"sha":"34306d94247469fb01aa946dcad6fd00a84f93b5","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"querier: delegate population stats to r_c_s","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/34306d94247469fb01aa946dcad6fd00a84f93b5"},{"sha":"c5f2fbb39669aee96446514987715e948ae39143","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"querier: manually inline find_querier()","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/c5f2fbb39669aee96446514987715e948ae39143"},{"sha":"6313de5d3f8d6dcf1c2f951904630d1a9f42af38","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"querier: store entries inside the r_c_s","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/6313de5d3f8d6dcf1c2f951904630d1a9f42af38"},{"sha":"ec6957a54e1345c7ad96362cf2b40458e08a9cc3","author":{"name":"Botond Dénes","email":"a584fa4d75a0a85d721f44fb6e06315a2853b0b6@scylladb.com"},"message":"reader_concurrency_semaphore: mandatate reads being stored in the inactive_read","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/ec6957a54e1345c7ad96362cf2b40458e08a9cc3"},{"sha":"9fa60f63ae38b5cc5f892c24b8fa470b901597d9","author":{"name":"Avi Kivity","email":"8f920f22884d6fea9df883843c4a8095a2e5ac6f@scylladb.com"},"message":"Merge \"Fix evictable shard reader related issues\" from Botond\n\n\"\nRecently some additional issues were discovered related to recent\nchanges to the way inactive readers are evicted and making shard readers\nevictable.\nOne such issue is that the `querier_cache` is not prepared for the\nquerier to be immediately evicted by the reader concurrency semaphore,\nwhen registered with it as an inactive read (#3987).\nThe other issue is that the multishard mutation query code was not\nfully prepared for evicted shard readers being re-created, or failing\nwhy being re-created (#3991).\n\nThis series fixes both of these issues and adds a unit test which covers\nthe second one. I am working on a unit test which would cover the second\nissue, but it's proving to be a difficult one and I don't want to delay\nthe fixes for these issues any longer as they also affect 3.0.\n\nFixes: #3987\nFixes: #3991\n\"\n\n* 'evictable-reader-related-issues/v1' of https://github.com/denesb/scylla:\n  reader_concurrency_semaphore: mandatate reads being stored in the inactive_read\n  querier: store entries inside the r_c_s\n  querier: manually inline find_querier()\n  querier: delegate population stats to r_c_s\n  querier: delegate expiry to reader_concurrency_semaphore\n  reader_concurrency_semaphore: add support for expiring inactive reads\n  querier: delegate memory limit enforcement to reader_concurrency_semaphore\n  reader_concurrency_semaphore: add memory limit\n  querier_cache: delegate resource based eviction stats to r_c_s\n  tests/querier_cache: prepare for counter migration\n  reader_concurrency_semaphore: make register_inactive_read a template\n  reader_concurrency_semaphore: make the inactive read handle a RAII object\n  treewide: prepare for a move-only inactive read handle\n  database: s/active_reads*/ongoing_reads*/\n  reader_concurrency_semaphore: add counters for inactive reads","distinct":true,"url":"https://api.github.com/repos/scylladb/scylla/commits/9fa60f63ae38b5cc5f892c24b8fa470b901597d9"}]}
{"actor":{"display_login":"avikivity"}}