Elon Gliksberg

Results 57 comments of Elon Gliksberg

Hi @manulpatel, thanks for reaching out. We're welcoming contributions from the community about this feature! Here's a guideline for the changes required if you're interested in contributing: 1. Add [`--log-format`...

Here's more information about this bug: 1. A new `WebviewProvider` is created on each Cline tab. 2. `WebviewProvider` creates a new `Controller`. 3. `Controller` creates `McpHub`. 4. Each `McpHub` tries...

> [@elongl](https://github.com/elongl) Can you show this happening with your processes? I've tested this before and confirmed that a new MCP server instance is spun up per webview, but then they...

Hi @aibazhang, thanks for opening this issue. Would you please care to elaborate on why you're using the `chunk` method instead of `max_query_size`? Asking because I think it'd make more...

@tsmo4 Have you tried lowering the [default `query_max_size`](https://github.com/elementary-data/dbt-data-reliability/blob/5f2ad803e76dcadf6e5d483c3d24705a015966f3/macros/edr/system/system_utils/get_config_var.sql#L76)?

Thanks a lot for this contribution, makes a lot of sense! Left one small comment, will merge afterwards.

Hi @kokorin, PR looks great, left one small comment in order to ensure the logic is backwards-compatible and doesn't break for users who haven't upgraded their dbt package yet. Thanks...

Hi @mstfkmlbsbdk2023! Thanks a lot for reporting this issue. We'll definitely make sure to update our docs in order to include the new experience with Microsoft Teams. For the time...

> Does the dispose method in controller/index.ts not do this? @celestial-vault It only gets called when a webview is closed, not when the app (VSCode) is terminated.

@celestial-vault @dcbartlett Resolved the conversation, feel free to merge.