search

datafuselabs / databend

𝗗𝗮𝘁𝗮, 𝗔𝗻𝗮𝗹𝘆𝘁𝗶𝗰𝘀 & 𝗔𝗜. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.29k stars 701 forks source link

refactor: Does the show procelist need to display ports? #15367

Closed TCeason closed 2 weeks ago

TCeason commented 2 weeks ago

Does the show procelist need to display ports?

Now the get client ip address will be obtained from the header. But this is a pure ip, and it's represented by a string.

In our current session ctx, client_ip is Option.

This appears to be CK-compatible behavior. But the port should be meaningless information.

So maybe we can set session _ctx.client_ip: Option and discard the port.

Originally posted by @TCeason in https://github.com/datafuselabs/databend/issues/15363#issuecomment-2081488983