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

Support multi column results with using mongo query runner #6558

Merged
merged 7 commits into from
Oct 29, 2023

Conversation

guidopetri
Copy link
Contributor

Identical contents to #6290 but with formatting fixes. I screwed up a rebase and accidentally closed the other PR irreversibly.

@guidopetri guidopetri changed the title Del zhenwu master Support multi column results with using mongo query runner Oct 28, 2023
@guidopetri
Copy link
Contributor Author

@justinclift would you kindly give me an approval on this since I can't do it myself (I'm the latest committer)? 😅

@codecov
Copy link

codecov bot commented Oct 28, 2023

Codecov Report

Merging #6558 (f189d53) into master (a5b01bf) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #6558      +/-   ##
==========================================
- Coverage   61.90%   61.89%   -0.01%     
==========================================
  Files         158      158              
  Lines       12966    12968       +2     
  Branches     1773     1772       -1     
==========================================
  Hits         8027     8027              
- Misses       4664     4665       +1     
- Partials      275      276       +1     
Files Coverage Δ
redash/query_runner/mongodb.py 40.38% <100.00%> (-0.40%) ⬇️

@justinclift
Copy link
Member

Just did the approve.

@guidopetri
Copy link
Contributor Author

Thanks mate :)

@guidopetri guidopetri merged commit 39e4ea1 into master Oct 29, 2023
15 of 16 checks passed
@guidopetri guidopetri deleted the del-zhenwu-master branch October 29, 2023 01:13
@guidopetri
Copy link
Contributor Author

@del-zhenwu thanks for these changes!

spacentropy added a commit to dodopizza/redash that referenced this pull request Jan 17, 2024
harveyrendell pushed a commit to pushpay/redash that referenced this pull request Jan 8, 2025
…#6558)

* Support multi column results with using mongo query runner

* format

Signed-off-by: del-zhenwu <[email protected]>

* Update test_mongodb.py

update unit test case: nested dict for mongodb runner

* Update test_mongodb.py

* fix formatting

---------

Signed-off-by: del-zhenwu <[email protected]>
Co-authored-by: del-zhenwu <[email protected]>
Co-authored-by: del-zhenwu <[email protected]>
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