Support --empty in satellite macro

[Korijn]

This fixes failures when running satellite macro with --empty:

WITH source_data AS (
      SELECT <columns>
      FROM (select * from <table> where false limit 0) _dbt_limit_subq_stg_area_v1 AS a
  ---------------------------------------------------------------------------------^^^
      WHERE <key> IS NOT NULL
  ),

With this PR the SQL becomes valid:

WITH source_data AS (
      SELECT <columns>
      FROM ((select * from <table> where false limit 0) _dbt_limit_subq_stg_area_v1) AS a
      WHERE <key> IS NOT NULL
  ),

[DVAlexHiggs]

Hello, thanks for this PR.

Can I ask if this only affects Databricks? I see you have only updated the Databricks implementation of the macro.

As for the PR itself, unfortunately, we’re not able to accept PRs directly to this repository at this time, as our active development repo is private.

I will look at re-producing this issue internally and testing it against all of our supported platforms as well.

If I find we can apply this fix, would you be happy for me to apply the change internally and credit you for it?

Thanks!

[Korijn]

Hello, thanks for this PR.

Can I ask if this only affects Databricks? I see you have only updated the Databricks implementation of the macro.

I have no idea to be honest, I don't know if other SQL dialects have the same requirements. I also have no means to test other providers.

As for the PR itself, unfortunately, we’re not able to accept PRs directly to this repository at this time, as our active development repo is private.

I will look at re-producing this issue internally and testing it against all of our supported platforms as well.

If I find we can apply this fix, would you be happy for me to apply the change internally and credit you for it?

I get that, no problem, this was just the most efficient way to communicate the change and rationale :)

I would love to see it applied! Credit would be amazing of course. :)

Thanks!

Thank you for the quick response!