feat(query): add max_query_row_nums

[FANNG1]

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

user may run a query select * from xx, this may scan too many rows. add a default limiter if user does't specify limit values.

Closes #issue

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated
databend	⬜️ Ignored (Inspect)		Dec 31, 2022 at 3:10AM (UTC)

[sundy-li]

better name to max_result_rows .

[BohuTANG]

@mergify update

[mergify]

update

✅ Branch has been successfully updated

[BohuTANG]

@sandflee

It would be great to add a SQL logic test for it so that we can keep the change always working :)

[FANNG1]

It would be great to add a SQL logic test for it so that we can keep the change always working :)

could u provide a example？

[BohuTANG]

Sure.
For example:

tests/sqllogictests/suites/base/20+_others/20_0011_setting

statement ok
DROP DATABASE IF EXISTS db1

statement ok
CREATE DATABASE db1

statement ok
USE db1

statement ok
CREATE TABLE IF NOT EXISTS t1(a INT) Engine = fuse

statement ok
INSERT INTO t1 (a) values (1), (2), (3)

query I
SELECT COUNT() FROM (SELECT * FROM t1)
----
3

statement ok
SET max_result_rows=1

query I
SELECT COUNT() FROM (SELECT * FROM t1)
----
1

statement ok
DROP TABLE t1

statement ok
DROP DATABASE db1

[BohuTANG]

2023