Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance duckdbExtension #2843

Merged
merged 2 commits into from
May 28, 2024
Merged

Conversation

PrateekGarg-gs
Copy link
Contributor

What type of PR is this?

What does this PR do / why is it needed ?

Which issue(s) this PR fixes:

Fixes #

Other notes for reviewers:

Does this PR introduce a user-facing change?

@PrateekGarg-gs PrateekGarg-gs requested a review from a team as a code owner May 10, 2024 12:40
Copy link

github-actions bot commented May 10, 2024

Test Results

     764 files       764 suites   1h 6m 25s ⏱️
12 617 tests 12 445 ✔️ 172 💤 0
15 728 runs  15 546 ✔️ 182 💤 0

Results for commit 7f40f67.

♻️ This comment has been updated with latest results.

@akphi
Copy link
Contributor

akphi commented May 27, 2024

@PrateekGarg-gs @gs-kotang can we merge change in #2836 into this PR? Thanks!
@aziemchawdhary-gs I want to merge these soon as they would help with DataCube PoC #2864 to improve the grid for REPL which is backed entirely by DuckDB for now.

Could we merge this?

@akphi akphi merged commit 13bc3b8 into finos:master May 28, 2024
4 checks passed
@akphi
Copy link
Contributor

akphi commented May 28, 2024

@aziemchawdhary-gs I rebased #2836 and merged that as well. Thanks @PrateekGarg-gs @gs-kotang @aziemchawdhary-gs !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants