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: Add docs for Amazon Dynamodb database driver #22145

Merged
merged 8 commits into from
Nov 18, 2022

Conversation

passren
Copy link
Contributor

@passren passren commented Nov 17, 2022

SUMMARY

Add docs for a new driver introduced for Amazon DynamoDB.
Updated dynamodb in the list of installing-database-drivers.mdx. Added a new page for PyDynamoDB driver.
This change associated with PR #22077.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Already run the doc in the local env.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@codecov
Copy link

codecov bot commented Nov 17, 2022

Codecov Report

Merging #22145 (5ea0832) into master (4496748) will decrease coverage by 0.10%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##           master   #22145      +/-   ##
==========================================
- Coverage   67.12%   67.02%   -0.11%     
==========================================
  Files        1818     1820       +2     
  Lines       69639    69567      -72     
  Branches     7496     7496              
==========================================
- Hits        46747    46627     -120     
- Misses      20954    21002      +48     
  Partials     1938     1938              
Flag Coverage Δ
hive 52.60% <70.58%> (-0.23%) ⬇️
mysql 78.12% <70.58%> (-0.28%) ⬇️
postgres 78.19% <70.58%> (-0.28%) ⬇️
presto 52.49% <70.58%> (-0.24%) ⬇️
python 81.38% <94.11%> (-0.19%) ⬇️
sqlite 76.65% <70.58%> (-0.28%) ⬇️
unit 50.89% <94.11%> (-0.31%) ⬇️

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

Impacted Files Coverage Δ
superset/db_engine_specs/dynamodb.py 94.11% <94.11%> (ø)
superset/tables/schemas.py 0.00% <0.00%> (-100.00%) ⬇️
superset/columns/schemas.py 0.00% <0.00%> (-100.00%) ⬇️
superset/tables/models.py 44.15% <0.00%> (-23.38%) ⬇️
superset/reports/notifications/slack.py 79.51% <0.00%> (-8.49%) ⬇️
superset/tasks/scheduler.py 79.62% <0.00%> (-2.38%) ⬇️
superset/reports/notifications/email.py 98.64% <0.00%> (-1.36%) ⬇️
superset/models/filter_set.py 77.27% <0.00%> (-1.19%) ⬇️
superset/connectors/sqla/utils.py 89.32% <0.00%> (-0.88%) ⬇️
superset/connectors/sqla/models.py 90.41% <0.00%> (-0.80%) ⬇️
... and 33 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@srinify srinify merged commit 9cebc1f into apache:master Nov 18, 2022
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 2.1.0 and removed 🚢 2.1.3 labels Mar 13, 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/L 🚢 2.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants