You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2024/03/04 20:32:19 UTC
(superset) branch test-ci updated (a34aec10cd -> fd9a6ec395)
This is an automated email from the ASF dual-hosted git repository.
michaelsmolina pushed a change to branch test-ci
in repository https://gitbox.apache.org/repos/asf/superset.git
discard a34aec10cd chore: test
omit e437356013 fix(tags): Improve support for tags with colons (#26965)
omit d8e26cfff1 chore: Remove database ID dependency for SSH Tunnel creation (#26989)
omit 43e1dc49c9 fix: bump FAB to 4.3.11 (#27039)
omit d5ef052044 build(deps): bump react-js-cron from 1.2.0 to 2.1.2 in /superset-frontend (#26981)
omit 9670572d99 build(deps-dev): bump copy-webpack-plugin from 9.1.0 to 12.0.2 in /superset-frontend (#26893)
omit 42b7bd5c03 fix(maps): france_regions.geojson generated with the notebook, from natural earth data (#27014)
omit 14f88e3f89 chore(sqllab): migrate to typescript (#26171)
omit 5d46d3a5d3 fix(actions): make tech debt uploader not block CI and skip w/o creds (#26966)
omit 5b34395689 fix: chart import validation (#26993)
omit 53daa1b13f chore(plugins): Description, Category and Tags for BigNumber with Period Time Comparison plugin (#27021)
omit e792460baf docs: add a note about database drivers in Docker builds (#27020)
omit 2cd7135a51 build(deps): bump @types/seedrandom from 2.4.30 to 3.0.8 in /superset-frontend (#26979)
omit edecf52ab9 chore(github): adding code owners for translation and country map wor… (#27000)
omit f79e08ec79 fix(cypress): resolving random dri3 error on cypress runner (#27001)
omit 607301ad56 fix(plugins): Fix dashboard filter in Period Over Period KPI plugin (#27013)
omit 7a270a5922 fix(helm): Fix inconsistency for the chart appVersion and default image tag (#27005)
omit 26025274a1 fix(maps): Move Overseas department and regions closer to France mainland (#26995)
omit 068bb719c8 docs: add notes to RELEASING about how to deploy docker images (#26998)
omit 32ce002296 build(deps): bump react-intersection-observer from 9.4.1 to 9.6.0 in /superset-frontend (#26996)
omit 92b52a7fa0 docs(presto): add Presto SSL connection details (#26986)
omit d8a9847503 fix: column values with NaN (#26946)
omit e874e04890 build(deps): bump @vx/legend from 0.0.198 to 0.0.199 in /superset-frontend/plugins/legacy-plugin-chart-histogram (#26526)
omit 80fd75dfe1 fix(ci): typo in my bash script (#26987)
omit db79c37707 fix(plugin): Period Over Period KPI Plugin Feature flag value (#26985)
omit 6b0d8ed265 feat: docker image tags documentation + tweaks (#26923)
omit fdff6d84f3 chore(dependencies): bump encodable to 0.7.8 (#26903)
omit a8cf443300 build(deps-dev): bump webpack from 5.90.0 to 5.90.1 in /docs (#26977)
omit 813783382c build(deps-dev): bump @types/node from 20.11.14 to 20.11.16 in /superset-websocket (#26974)
omit 5930bbb5d3 build(deps): bump actions/checkout from 2 to 4 (#26971)
omit 760affeb13 build(deps): bump actions/cache from 1 to 4 (#26972)
omit a8a9472c39 build(deps): bump actions/setup-python from 4 to 5 (#26970)
omit 90b6ba0a4d chore(actions): getting fancier with labels (#26950)
omit b26f82e311 fix(ci): support action/script v5 breaking change v2 (#26969)
omit e6d2fb6fdf fix(plugin-chart-table): Prevent misalignment of totals and headers when scrollbar is visible (#26964)
omit b33662f6ff build(deps): bump actions/setup-java from 1 to 4 (#26952)
omit c6d809a2a4 fix(ci): support action/script v5 breaking change (#26968)
omit e4eae9a70c fix(plugin-chart-table): Revert "fix(chart table in dashboard): improve screen reading of table (#26453)" (#26963)
omit fb38552a14 build(deps-dev): bump mock-socket from 9.0.3 to 9.3.1 in /superset-frontend (#26958)
omit 56069b05f9 fix(cache): remove unused webserver config & handle trailing slashes (#22849)
omit 4c5176eea8 fix: Allow exporting saved queries without schema information (#26889)
omit 4c245850c4 build(deps): bump actions/github-script from 3 to 7 (#26953)
omit 26e330e119 feat(ci): kill duplicate CI jobs on PRs (#26945)
omit 032af5a069 build(deps): bump actions/setup-node from 2 to 4 (#26927)
omit 83acac3e43 fix(actions): specify branch on monorepo lockfile pusher (#26949)
omit 5ed8f6784f build(deps): bump aws-actions/configure-aws-credentials from 1 to 4 (#26954)
omit 3f7a1f68ff build(deps): bump aws-actions/amazon-ecr-login from 1 to 2 (#26955)
omit b7ddfff0a1 build(deps): bump github/codeql-action from 2 to 3 (#26956)
omit 36ce9e26f0 fix: dashboard import validation (#26887)
omit 0124a5c055 build(deps): bump moment from 2.29.4 to 2.30.1 in /superset-frontend (#26938)
new fd9a6ec395 chore: test
This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version. This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:
* -- * -- B -- O -- O -- O (a34aec10cd)
\
N -- N -- N refs/heads/test-ci (fd9a6ec395)
You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.
Any revisions marked "omit" are not gone; other references still
refer to them. Any revisions marked "discard" are gone forever.
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.github/CODEOWNERS | 7 -
.github/dependabot.yml | 29 +
.github/labeler.yml | 128 ----
.github/workflows/bashlib.sh | 6 +-
.github/workflows/cancel_duplicates.yml | 2 +-
.github/workflows/check_db_migration_confict.yml | 9 +-
.github/workflows/chromatic-master.yml | 2 +-
.github/workflows/codeql-analysis.yml | 11 +-
.github/workflows/dependency-review.yml | 2 +-
.github/workflows/docker-release.yml | 43 +-
.github/workflows/embedded-sdk-release.yml | 4 +-
.github/workflows/embedded-sdk-test.yml | 9 +-
.github/workflows/ephemeral-env-pr-close.yml | 8 +-
.github/workflows/ephemeral-env.yml | 47 +-
.github/workflows/generate-FOSSA-report.yml | 7 +-
.github/workflows/labeler.yml | 9 -
.github/workflows/latest-release-tag.yml | 2 +-
.github/workflows/license-check.yml | 12 +-
.github/workflows/no-hold-label.yml | 7 +-
.github/workflows/no-op.yml | 14 +
.github/workflows/pr-lint.yml | 2 +-
.github/workflows/pre-commit.yml | 9 +-
.github/workflows/prefer-typescript.yml | 8 +-
.github/workflows/release.yml | 8 +-
.github/workflows/superset-applitool-cypress.yml | 6 +-
.../workflows/superset-applitools-storybook.yml | 4 +-
.github/workflows/superset-cli.yml | 9 +-
.github/workflows/superset-docs-deploy.yml | 4 +-
.github/workflows/superset-docs-verify.yml | 10 +-
.github/workflows/superset-e2e.yml | 8 +-
.github/workflows/superset-frontend.yml | 9 +-
.github/workflows/superset-helm-lint.yml | 9 +-
.github/workflows/superset-helm-release.yml | 2 +-
.../workflows/superset-python-integrationtest.yml | 12 +-
.github/workflows/superset-python-misc.yml | 13 +-
.github/workflows/superset-python-presto-hive.yml | 13 +-
.github/workflows/superset-python-unittest.yml | 9 +-
.github/workflows/superset-translations.yml | 13 +-
.github/workflows/superset-websocket.yml | 7 +-
.github/workflows/tech-debt.yml | 23 +-
.github/workflows/update-monorepo-lockfiles.yml | 12 +-
CONTRIBUTING.md | 4 +-
RELEASING/README.md | 14 -
docker-compose-non-dev.yml | 2 +-
docs/README.md | 5 +-
docs/docs/databases/presto.mdx | 11 -
docs/docs/installation/docker.mdx | 77 --
docs/package.json | 2 +-
docs/yarn.lock | 8 +-
helm/superset/Chart.yaml | 2 +-
helm/superset/README.md | 4 +-
helm/superset/values.yaml | 2 +-
requirements/base.txt | 19 +-
requirements/development.txt | 13 +-
requirements/integration.txt | 6 -
scripts/build_docker.py | 25 +-
setup.py | 2 +-
superset-frontend/package-lock.json | 828 +++++++--------------
superset-frontend/package.json | 10 +-
.../packages/superset-ui-core/package.json | 2 +-
.../superset-ui-core/src/query/types/Query.ts | 1 -
.../superset-ui-core/src/utils/featureFlags.ts | 2 +-
.../scripts/Country Map GeoJSON Generator.ipynb | 441 +++--------
.../src/countries/france.geojson | 9 +-
.../src/countries/france_regions.geojson | 34 +-
.../src/countries_custom/france_regions.geojson | 20 +
.../legacy-plugin-chart-histogram/package.json | 2 +-
.../legacy-preset-chart-deckgl/package.json | 2 +-
.../plugins/legacy-preset-chart-nvd3/package.json | 2 +-
.../plugins/plugin-chart-echarts/package.json | 2 +-
.../src/plugin/buildQuery.ts | 16 +-
.../src/plugin/index.ts | 11 +-
.../src/plugin/transformProps.ts | 12 +-
.../plugin-chart-table/src/DataTable/DataTable.tsx | 1 +
.../src/DataTable/hooks/useSticky.tsx | 25 +-
.../plugins/plugin-chart-table/src/Styles.tsx | 7 +-
.../plugins/plugin-chart-word-cloud/package.json | 2 +-
superset-frontend/spec/helpers/reducerIndex.ts | 4 +-
.../components/App/{App.test.tsx => App.test.jsx} | 31 +-
.../SqlLab/components/App/{index.tsx => index.jsx} | 52 +-
.../src/SqlLab/components/SqlEditor/index.tsx | 5 +-
...rLeftBar.test.tsx => SqlEditorLeftBar.test.jsx} | 177 ++---
.../SqlLab/components/SqlEditorLeftBar/index.tsx | 23 +-
.../TabbedSqlEditors/TabbedSqlEditors.test.jsx | 228 ++++++
.../TabbedSqlEditors/TabbedSqlEditors.test.tsx | 178 -----
.../TabbedSqlEditors/{index.tsx => index.jsx} | 95 +--
superset-frontend/src/SqlLab/fixtures.ts | 2 +-
superset-frontend/src/SqlLab/types.ts | 19 +-
...ryResults.test.ts => emptyQueryResults.test.js} | 7 +-
...Helper.ts => reduxStateToLocalStorageHelper.js} | 18 +-
.../src/components/EmptyState/index.tsx | 2 +-
superset-frontend/src/components/Tags/utils.tsx | 9 +-
superset-frontend/src/features/tags/tags.ts | 16 +-
superset-frontend/src/types/bootstrapTypes.ts | 1 -
superset-websocket/package-lock.json | 14 +-
superset-websocket/package.json | 2 +-
superset/commands/chart/importers/v1/utils.py | 13 +-
superset/commands/dashboard/importers/v1/utils.py | 14 +-
superset/commands/database/create.py | 58 +-
superset/commands/database/ssh_tunnel/create.py | 23 +-
superset/commands/database/update.py | 2 +-
superset/commands/dataset/importers/v1/utils.py | 7 +-
superset/commands/query/export.py | 11 +-
superset/config.py | 4 +
superset/errors.py | 1 -
superset/jinja_context.py | 9 +-
superset/models/helpers.py | 7 +-
superset/security/manager.py | 41 -
superset/tasks/cache.py | 4 +-
superset/utils/core.py | 9 -
tests/integration_tests/charts/commands_tests.py | 2 +-
tests/integration_tests/conftest.py | 21 +-
.../integration_tests/dashboards/commands_tests.py | 2 +-
tests/integration_tests/databases/api_tests.py | 16 +-
.../ssh_tunnel/commands/commands_tests.py | 17 +
tests/integration_tests/datasets/commands_tests.py | 2 +-
tests/integration_tests/datasource/api_tests.py | 10 -
tests/integration_tests/datasource_tests.py | 9 +-
tests/integration_tests/superset_test_config.py | 1 +
.../superset_test_config_thumbnails.py | 1 +
tests/integration_tests/tasks/test_cache.py | 58 --
.../charts/commands/importers/v1/import_test.py | 164 +---
.../commands/importers/v1/import_test.py | 150 +---
.../databases/ssh_tunnel/commands/create_test.py | 6 +-
tests/unit_tests/scripts/docker_build.py | 14 +-
tests/unit_tests/security/manager_test.py | 140 ----
126 files changed, 1279 insertions(+), 2593 deletions(-)
delete mode 100644 docs/docs/installation/docker.mdx
create mode 100644 superset-frontend/plugins/legacy-plugin-chart-country-map/src/countries_custom/france_regions.geojson
rename superset-frontend/src/SqlLab/components/App/{App.test.tsx => App.test.jsx} (86%)
rename superset-frontend/src/SqlLab/components/App/{index.tsx => index.jsx} (85%)
rename superset-frontend/src/SqlLab/components/SqlEditorLeftBar/{SqlEditorLeftBar.test.tsx => SqlEditorLeftBar.test.jsx} (54%)
create mode 100644 superset-frontend/src/SqlLab/components/TabbedSqlEditors/TabbedSqlEditors.test.jsx
delete mode 100644 superset-frontend/src/SqlLab/components/TabbedSqlEditors/TabbedSqlEditors.test.tsx
rename superset-frontend/src/SqlLab/components/TabbedSqlEditors/{index.tsx => index.jsx} (83%)
rename superset-frontend/src/SqlLab/utils/{emptyQueryResults.test.ts => emptyQueryResults.test.js} (94%)
rename superset-frontend/src/SqlLab/utils/{reduxStateToLocalStorageHelper.ts => reduxStateToLocalStorageHelper.js} (83%)
delete mode 100644 tests/integration_tests/tasks/test_cache.py
(superset) 01/01: chore: test
Posted by mi...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
michaelsmolina pushed a commit to branch test-ci
in repository https://gitbox.apache.org/repos/asf/superset.git
commit fd9a6ec3953f26d43a1a851aa290782c2f679295
Author: Michael S. Molina <mi...@gmail.com>
AuthorDate: Mon Mar 4 15:32:06 2024 -0500
chore: test
---
CONTRIBUTING.md | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index e95a766dc7..998dd48e0d 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -40,7 +40,7 @@ little bit helps, and credit will always be given.
- [Authoring](#authoring)
- [Reviewing](#reviewing)
- [Test Environments](#test-environments)
- - [Merging](#merging)
+ - [Mergings](#merging)
- [Post-merge Responsibility](#post-merge-responsibility)
- [Design Guidelines](#design-guidelines)
- [Capitalization guidelines](#capitalization-guidelines)
@@ -217,7 +217,7 @@ Security team members must:
- Have an [ICLA](https://www.apache.org/licenses/contributor-agreements.html) signed with Apache Software Foundation.
- Not reveal information about pending and unfixed security issues to anyone (including their employers) unless specifically authorised by the security team members, e.g., if the security team agrees that diagnosing and solving an issue requires the involvement of external experts.
-A release manager, the contributor overseeing the release of a specific version of Apache Superset, is by default a member of the security team. However, they are not expected to be active in assessing, discussing, and fixing security issues.
+A release manager, the contributor overseeing the release of a specific version of Apache Superset, is by default a member of the security team. However, they are not expected to be active in assessing, discussing, and fixing security issues.
Security team members should also follow these general expectations:
@@ -374,8 +374,8 @@ Triaging goals
First, add **Category labels (a.k.a. hash labels)**. Every issue/PR must have one hash label (except spam entry). Labels that begin with `#` defines issue/PR type:
-| Label | for Issue | for PR |
-| --------------- | ----------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------- |
+| Label | for Issue | for PR |
+| --------------- | ----------------------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------- |
| `#bug` | Bug report | Bug fix |
| `#code-quality` | Describe problem with code, architecture or productivity | Refactor, tests, tooling |
| `#feature` | New feature request | New feature implementation |
@@ -648,22 +648,28 @@ If while using the above commands you encounter an error related to the limit of
```bash
Error: ENOSPC: System limit for number of file watchers reached
```
+
The error is thrown because the number of files monitored by the system has reached the limit.
You can address this this error by increasing the number of inotify watchers.
The current value of max watches can be checked with:
+
```bash
cat /proc/sys/fs/inotify/max_user_watches
```
+
Edit the file /etc/sysctl.conf to increase this value.
The value needs to be decided based on the system memory [(see this StackOverflow answer for more context)](https://stackoverflow.com/questions/535768/what-is-a-reasonable-amount-of-inotify-watches-with-linux).
Open the file in editor and add a line at the bottom specifying the max watches values.
+
```bash
fs.inotify.max_user_watches=524288
```
+
Save the file and exit editor.
To confirm that the change succeeded, run the following command to load the updated value of max_user_watches from sysctl.conf:
+
```bash
sudo sysctl -p
```