Skip to content
New issue

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

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

Already on GitHub? # to your account

docs: Update Dremio doc with Arrow Flight connection details #11422

Merged
merged 2 commits into from
Oct 26, 2020

Conversation

narendrans
Copy link
Contributor

SUMMARY

  • Add connection string via Arrow Flight
  • Modify the connector link

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen Shot 2020-10-25 at 8 04 50 PM

* Add connection string via Arrow Flight
* Modify the connector link
@narendrans narendrans changed the title update dremio doc w arrow flight connection Update Dremio doc with Arrow Flight connection details Oct 26, 2020
@narendrans narendrans changed the title Update Dremio doc with Arrow Flight connection details docs: Update Dremio doc with Arrow Flight connection details Oct 26, 2020
@codecov-io
Copy link

codecov-io commented Oct 26, 2020

Codecov Report

Merging #11422 into master will decrease coverage by 6.49%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11422      +/-   ##
==========================================
- Coverage   66.58%   60.08%   -6.50%     
==========================================
  Files         860      823      -37     
  Lines       40874    39508    -1366     
  Branches     3694     3477     -217     
==========================================
- Hits        27215    23738    -3477     
- Misses      13562    15604    +2042     
- Partials       97      166      +69     
Flag Coverage Δ
#cypress 56.58% <ø> (-0.06%) ⬇️
#javascript ?
#python 61.93% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...uperset-frontend/src/dashboard/util/dnd-reorder.js 0.00% <0.00%> (-100.00%) ⬇️
...rset-frontend/src/dashboard/util/getEmptyLayout.js 0.00% <0.00%> (-100.00%) ⬇️
...et-frontend/src/components/Menu/LanguagePicker.tsx 0.00% <0.00%> (-100.00%) ⬇️
...dashboard/components/resizable/ResizableHandle.jsx 0.00% <0.00%> (-100.00%) ⬇️
.../src/dashboard/util/getFilterScopeFromNodesTree.js 0.00% <0.00%> (-93.48%) ⬇️
...uperset-frontend/src/utils/getClientErrorObject.ts 0.00% <0.00%> (-89.19%) ⬇️
.../src/dashboard/components/FilterIndicatorGroup.jsx 11.76% <0.00%> (-88.24%) ⬇️
...c/explore/components/controls/withVerification.jsx 9.09% <0.00%> (-87.88%) ⬇️
...src/dashboard/components/gridComponents/Header.jsx 10.52% <0.00%> (-86.85%) ⬇️
...rset-frontend/src/profile/components/Favorites.tsx 0.00% <0.00%> (-86.67%) ⬇️
... and 265 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8aecffd...bd4e56b. Read the comment docs.

@@ -9,13 +9,18 @@ version: 1
## Dremio

The recommended connector library for Dremio is
[sqlalchemy_dremio](https://github.com/sqggles/sqlalchemy_dremio).
[sqlalchemy_dremio](https://github.com/narendrans/sqlalchemy_dremio).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, can we make the link point to the PyPI page?

Copy link
Contributor Author

@narendrans narendrans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@villebro villebro merged commit 2e6afdf into apache:master Oct 26, 2020
auxten pushed a commit to auxten/incubator-superset that referenced this pull request Nov 20, 2020
…11422)

* update dremio doc w arrow flight connection

* Add connection string via Arrow Flight
* Modify the connector link

* Update dremio.mdx
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.0.0 labels Mar 12, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/XS 🚢 1.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants