databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.85k stars 750 forks source link

feat(query): Procedure Part2 support arguments #16453

Closed TCeason closed 1 month ago

TCeason commented 2 months ago

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

Summary

Parat II of https://github.com/datafuselabs/databend/pull/16348

this pr support call procedure();

After this pr, we need to add a new issue to enhanced procedure checks to make it more of a product than a prototype

fixes: Part of https://github.com/datafuselabs/databend/issues/14904

Tests

Type of change


This change isโ€‚Reviewable

TCeason commented 2 months ago

Why is args special and is converted to lower cases?

No need to modify it. Already convert to lower in parse create/drop procedure.

e.g.

create procedure p1(x UInt8) -> should can be drop procedure p1(uint8)

TCeason commented 2 months ago

warit https://github.com/datafuselabs/databend/pull/16455 merged.

sundy-li commented 1 month ago

docs cc @soyeric128 with @TCeason