adap-435 Use explicit begin transaction

[shreyadatar]

resolves #556
docs dbt-labs/docs.getdbt.com/#

Problem

Mixing Snowflake begin/end scripting blocks with transactions throws a syntax error.

Solution

By using the more explicit begin transaction instead of begin, the above confusion and syntax error is avoided, therefore allowing both begin/end blocks as well as transactions being used together.

Checklist

• I have read the contributing guide and understand what's expected of me
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

[shreyadatar]

Hi @colin-rogers-dbt, looks like this failed a few CI checks. Is there a follow-up TODO here?

[lokofoko]

Any updates here? The issue is still not solved and it is impossible to create a procedure in snowflake from dbt.

[amychen1776]

@lokofoko Could I ask for your use case for using stored procedures with dbt? Stored procedures are generally anti-pattern to our dbt best practices.

[amychen1776]

Due to a lack of response and also dbt not recommending stored procedures - I'm going to close out this PR for now.