Updated Dialect Variable Name

databrickslabs / remorph

Cross-compiler and Data Reconciler into Databricks Lakehouse

Other

21 stars 13 forks source link

Updated Dialect Variable Name #535

Closed vijaypavann-db closed 1 month ago

vijaypavann-db commented 1 month ago

In the QueryBuilder class, the Dialect instance variable name is referred as source which is incorrect as Dialect refers to Source / Target as per the initialization.

Updated Dialect variable name from source to engine.
Removed unnecessary paranthesis (list)

github-actions[bot] commented 1 month ago

Coverage tests results

394 tests ±0 357 :white_check_mark: ±0 4s :stopwatch: ±0s 2 suites ±0 2 :zzz: ±0 2 files ±0 35 :x: ±0

For more details on these failures, see this check.

Results for commit 227880b5. ± Comparison against base commit 440fd48d.

:recycle: This comment has been updated with latest results.

sundarshankar89 commented 1 month ago

LGTM