Mikhail f. Shiryaev

Results 366 comments of Mikhail f. Shiryaev

Dear @EnricoMi, I am not sure regarding the `totalCount` implementation here. The `PaginatedList` class requests URLs here and there, and it feels like either the `incomplete_results` should be set everywhere...

Hey-hey dear @EnricoMi, is there a way I can help to get this change into a released version? We're very badly affected by the lack of `incomplete_results` flag in PaginatedList...

Hi-hi! I'd like to get the first review round, please :pray:

Thanks for the PR! Looks like the tests has failed, unfortunately =\

I wrote a test, unfortunately, it fails as following: ``` =================================== FAILURES =================================== _______________________________ test_ttl_version _______________________________ [gw0] linux -- Python 3.10.12 /usr/bin/python3 test_graphite_merge_tree/test.py:534: in test_ttl_version q("OPTIMIZE TABLE test.graphite PARTITION 200109...

I can only approve that the original issue that we found in https://github.com/ClickHouse/ClickHouse/issues/88002 is addressed, since my tests are OK :+1:

The following new stack trace from CI Logs `system.crash_log` found: ``` DB::abortOnFailedAssertion(String const&, void* const*, unsigned long, unsigned long) DB::handle_error_code(String const&, std::basic_string_view, int, bool, std::vector const&) DB::Exception::Exception(DB::Exception::MessageMasked&&, int, bool) Exception...

We need to think of parametric `pastila.py` to cover other endpoints. I will cover it within the scope of this PR.

``` > pacman -Qi python-cryptography | ./pastila.py https://pastila.nl/?06252529/dea1d129db4f8ac5cc2cf319a9336868#O6bLQva3VEr6KMw64Yf41g== > ./pastila.py https://pastila.nl/?06252529/dea1d129db4f8ac5cc2cf319a9336868#O6bLQva3VEr6KMw64Yf41g== Name : python-cryptography Version : 45.0.5-1 Description : A package designed to expose cryptographic recipes and primitives to Python...