Ilya Golshtein

> Please resolve conflicts. Hello @alexey-milovidov, looks like I've failed to catch master again ;) But I've done some performance tests https://github.com/ClickHouse/ClickHouse/issues/34236#issuecomment-1229920651

> Tests are here https://github.com/ClickHouse/ClickHouse/pull/31796/files#diff-e017b5f2830a8e578431a0216d0442c2076d0994ce31303b40e6f7346d318fe4R63. In this pull request JOIN planning is analyze and planning stages are completely reimplemented. Main changes related to JOINs: > > 1. Multiple JOINs are...

To me, degradation of queries like 'SELECT count() FROM numbers(50000000) WHERE NOT ignore(toDate('2017-01-01') + number % 1000 + rand() % 10 AS t, toRelativeDayNum(t))' from https://s3.amazonaws.com/clickhouse-test-reports/39425/ac2a42cfb0f68252d0fb2fcf8b17ec70949c5566/performance_comparison_[4/4]/report.html is meaningful (e.g. because...

Thanks @rvasin , very interesting. Looks like the investigation can be not as simple and comfortable as I thought, but it is required anyway. We see serious degradation in some...

I've tried to benchmark normalizeDayNum via quickbench https://quick-bench.com/q/gSrrRikIlfAo4S2Nb9EzrqLWqKE . Totally pointless: there is no difference. But funny. Still don't understand the performance difference between runs, branch predictor hardly can work...

The main reason to not make a PR is intention to prevent this change from going to master as is instead of proper refactoring.

Have it in prod. Several months. So far, so good. No memory leaks or unpleasant surprises of any kind. Management and monitoring purposes, so request rate is fairly low.

Is amalgamation required condition to observe this issue? IOW, is anyone who does not use amalgamation seeing this?

A quick glance reveals (happy to make a mistake) that HTTP method is not a part of a route and it does not seem easy to alter this behavior. It...

Sure, right. The question is if it is possible to have different handlers for different HTTP methods and same URL. In flask one can have ```python @app.route('/api/v1/table', methods=['GET']) def test():...