feat(python): add sql() and show() convenience method to QueryBuilder

[DevChrisCross]

Description

Adds user friendly methods, namely sql() and show() to the QueryBuilder

Related Issue(s)

• closes Feature request : user friendly methods for Querybuilder , show and sql #3076

Documentation

[github-actions]

ACTION NEEDED

delta-rs follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.51%. Comparing base (6430151) to head (a420c33).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3100      +/-   ##
==========================================
- Coverage   72.52%   72.51%   -0.01%     
==========================================
  Files         128      128              
  Lines       41201    41201              
  Branches    41201    41201              
==========================================
- Hits        29882    29878       -4     
- Misses       9408     9412       +4     
  Partials     1911     1911              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ion-elgreco]

@DevChrisCross the .SQL() should rather give an object that's lazy executed, similar to duckdb's python API:

https://duckdb.org/docs/api/python/reference/

You can see the method .SQL() and .show() there

[djouallah]

for what is worth it, instead of qb.show().execute(query)
it should be
qb.execute(query).show()

[DevChrisCross]

I've introduced a QueryResult class to accommodate the lazy execution, and in turn, introduced fetchall() as well, let me know if it is okay @ion-elgreco @djouallah

[djouallah]

do you mind adding qb.sql(query).show() too please

[DevChrisCross]

do you mind adding qb.sql(query).show() too please

Ah yes, that was already added as well :)

[ion-elgreco]

@DevChrisCross looks good, perhaps also add a to_arrow() which returns a pyarrow table and to_pandas() on it as well : )

[rtyler]

I am squashing this and will then merge, I love seeing more improvements here 😄

[DevChrisCross]

@rtyler Oh it was merged already? I was initially trying to accommodate comments @ion-elgreco regarding the to_arrow and to_pandas methods 😅 Is it still needed?

[rtyler]

@DevChrisCross I think those are good ideas to add in additional pull requests 😄