SNOW-1882588 Allow Empty Sql Text when Dataframe Ast is Presented

[sfc-gh-bli]

Please answer these questions before submitting your pull requests. Thanks!

1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

   SNOW-1882588 Allow Empty Sql Text when Dataframe Ast is Presented

2. Fill out the following pre-review checklist:

   • I am adding a new automated test(s) to verify correctness of my new code
   • I am adding new logging messages
   • I am adding a new telemetry message
   • I am modifying authorization mechanisms
   • I am adding new credentials
   • I am modifying OCSP code
   • I am adding a new dependency

3. Please describe how your code solves the related issue.

   Please write a short description of how your code change solves the related issue.

4. (Optional) PR for stored-proc connector:

[github-actions]

Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

---

I have read the CLA Document and I hereby sign the CLA

---

_{You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.}

[sfc-gh-bli]

Change None to "", otherwise the following log formatters will fail.

[sfc-gh-lspiegelberg]

that makes sense.

[sfc-gh-lspiegelberg]

Can we maybe clarify the logging logic here? If I understand this correctly, basically in the case of _dataframe_ast =True, command will be "". Should we log out the base64 string in debug mode? I.e.,

if not command:
   if _dataframe_ast:
         logger.debug("query[AST base64]: [%s]", _dataframe_ast)
   else:
         logger.warning("execute: no query is given to execute")
         return      

[sfc-gh-bli]

Good Idea, let me change it.

[sfc-gh-bli]

changed

[sfc-gh-lspiegelberg]

lgtm, some comment re logging. I think we can clear this up a bit.

[sfc-gh-mkeller]

Please add tests for this change!

[sfc-gh-bli]

I have read the CLA Document and I hereby sign the CLA