rusackas closed pull request #27047: feat(actions): auto-label PRs that close
issues.
URL: https://github.com/apache/superset/pull/27047
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
enricozab commented on issue #29825:
URL: https://github.com/apache/superset/issues/29825#issuecomment-2266357294
I'm using Windows 11 and Docker Desktop with WSL2. I have already set the
memory to 12GB (even tried up to 16GB) but I still get stuck at that part. It
doesn't crash, it's just
rtexelm merged PR #29674:
URL: https://github.com/apache/superset/pull/29674
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@s
sadpandajoe commented on PR #29846:
URL: https://github.com/apache/superset/pull/29846#issuecomment-2266301927
@supersetbot label 4.1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
eschutho opened a new pull request, #29846:
URL: https://github.com/apache/superset/pull/29846
### SUMMARY
Loading slack channels earlier to ensure that the method dropdowns have the
correct value when they are loaded.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
rtexelm merged PR #29843:
URL: https://github.com/apache/superset/pull/29843
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@s
rtexelm merged PR #29845:
URL: https://github.com/apache/superset/pull/29845
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@s
mistercrunch closed pull request #29506: fix: Download to CSV shows a special
Chinese characters
URL: https://github.com/apache/superset/pull/29506
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to th
mistercrunch commented on PR #29506:
URL: https://github.com/apache/superset/pull/29506#issuecomment-2266265591
Closing for now, happy to push Werkzeug dependency forward when the fix is in
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
mistercrunch commented on PR #29586:
URL: https://github.com/apache/superset/pull/29586#issuecomment-2266264490
Somehow conflicting with something, happy to semi-blindly merge if/when
conflicts are resolved
--
This is an automated message from the Apache Git Service.
To respond to the mes
mistercrunch merged PR #29808:
URL: https://github.com/apache/superset/pull/29808
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr
mistercrunch commented on PR #29844:
URL: https://github.com/apache/superset/pull/29844#issuecomment-2266260543
--
This is an automated message from the Apache Git Service.
To respond to the mess
mistercrunch commented on PR #29686:
URL: https://github.com/apache/superset/pull/29686#issuecomment-2266258007
Yup here's a link to it for reference.
https://github.com/apache/superset/blob/master/superset-frontend/src/components/DropdownButton/index.tsx#L38
Curious why it's there
mistercrunch commented on issue #27416:
URL: https://github.com/apache/superset/issues/27416#issuecomment-2266254267
I think drill to details adds in "Calculated Columns" defined in at the
dataset level and will fail if there's an error in any of them. I just hit a
similar issue and deletin
rtexelm opened a new pull request, #29845:
URL: https://github.com/apache/superset/pull/29845
### SUMMARY
Convert tests to RTL
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
### TESTING INSTRUCTIONS
Run tests
### ADDITIONAL INFORMATION
-
rusackas opened a new pull request, #29844:
URL: https://github.com/apache/superset/pull/29844
### SUMMARY
I'm running this locally, and I don't see any trouble yet. Let's see if it
passes CI!
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
### TESTING INSTRUCTI
villebro merged PR #29591:
URL: https://github.com/apache/superset/pull/29591
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@
rtexelm opened a new pull request, #29843:
URL: https://github.com/apache/superset/pull/29843
### SUMMARY
Converts the current tests to use RTL
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
### TESTING INSTRUCTIONS
Run tests
### ADDITIONAL INFORMA
rusackas opened a new pull request, #29842:
URL: https://github.com/apache/superset/pull/29842
### SUMMARY
Just experimentally bumping bootstrap, to reduce CVEs. No idea if this will
work, but will check on an ephemeral.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
villebro commented on PR #28706:
URL: https://github.com/apache/superset/pull/28706#issuecomment-2266208909
@rmasters this needs a rebase, and I don't have permissions to push changes
to your branch to do it. Can you rebase, and optionally make that `utils`
refactor?
--
This is an automa
villebro commented on code in PR #28706:
URL: https://github.com/apache/superset/pull/28706#discussion_r1702365181
##
superset/tasks/cache.py:
##
@@ -212,14 +212,38 @@ def get_payloads(self) -> list[dict[str, int]]:
strategies = [DummyStrategy, TopNDashboardsStrategy, Dashboard
rusackas opened a new pull request, #29841:
URL: https://github.com/apache/superset/pull/29841
### SUMMARY
Picking this up from the stale PR here
https://github.com/apache/superset/pull/18057
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
### TESTING INSTRUCTIO
betodealmeida opened a new pull request, #29840:
URL: https://github.com/apache/superset/pull/29840
### SUMMARY
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
### TESTING INSTRUCTIONS
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
-
villebro opened a new issue, #29839:
URL: https://github.com/apache/superset/issues/29839
## [SIP] Proposal for Global Async Task Framework
### Motivation
*Note*: This replaces [[SIP-141] Global Async Queries 2.0](#29515) which
aimed at completing [[SIP-39] Global Async Query S
rusackas merged PR #29838:
URL: https://github.com/apache/superset/pull/29838
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@
eschutho merged PR #29721:
URL: https://github.com/apache/superset/pull/29721
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@
michael-s-molina commented on issue #29546:
URL: https://github.com/apache/superset/issues/29546#issuecomment-2265853714
> PRs introducing database migrations must include runtime estimates and
downtime expectations.
@mistercrunch that's what I'm currently doing while testing 4.1. If
mistercrunch commented on issue #29825:
URL: https://github.com/apache/superset/issues/29825#issuecomment-2265848286
Does it crash/gets killed? Seems memory-related too or more inderectly
swapping if it's just coming to a crawl.
We'll need a way to control/diminish memory usage when b
mistercrunch commented on issue #29546:
URL: https://github.com/apache/superset/issues/29546#issuecomment-2265816516
Oh I just found prior art here
https://github.com/apache/superset/issues/13351
> PRs introducing database migrations must include runtime estimates and
downtime expect
rusackas commented on issue #29546:
URL: https://github.com/apache/superset/issues/29546#issuecomment-2265812261
> @michael-s-molina I added the section "Considerations for large
environments:". I'm removing the DRAFT tag and will open a PR setting us up as
CODEOWNERS
There's also a
mistercrunch commented on issue #29546:
URL: https://github.com/apache/superset/issues/29546#issuecomment-2265807184
@michael-s-molina I added the section "Considerations for large
environments:". I'm removing the DRAFT tag and will open a PR setting us up as
CODEOWNERS
--
This is an aut
rusackas commented on issue #29835:
URL: https://github.com/apache/superset/issues/29835#issuecomment-2265796605
This is a completely wild and uneducated guess, but does `from
s3cache.s3cache import S3Cache` happen to work? That's what it looks like in
[these docs](https://pypi.org/project/
betodealmeida commented on code in PR #29802:
URL: https://github.com/apache/superset/pull/29802#discussion_r1702099935
##
superset/db_engine_specs/databricks.py:
##
@@ -434,7 +434,25 @@ def get_default_catalog(
cls,
database: Database,
) -> str | None:
+
mistercrunch commented on issue #29546:
URL: https://github.com/apache/superset/issues/29546#issuecomment-2265793204
Relevant:
https://github.com/apache/superset/issues/29801#issuecomment-2263052989 -
posting this here and editing the SIP to tackle this type of issue
--
This is an automa
betodealmeida commented on code in PR #29802:
URL: https://github.com/apache/superset/pull/29802#discussion_r1702090563
##
pyproject.toml:
##
@@ -236,7 +236,7 @@ legacy_tox_ini = """
# Remember to start celery workers to run celery tests, e.g.
# celery --app=superset.tasks.cel
villebro commented on issue #29515:
URL: https://github.com/apache/superset/issues/29515#issuecomment-2265732362
We are closing this SIP and will be opening a new one to reflect a new
direction that was established during discussions. But in summary:
- WebSockets will remain as is, pollin
villebro closed issue #29515: [SIP-141] Simplified Global Async Queries
URL: https://github.com/apache/superset/issues/29515
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
hainenber commented on PR #29652:
URL: https://github.com/apache/superset/pull/29652#issuecomment-2265713197
`npm install`-ed but sadly the CI had a Chrome crash during 1st E2E :(
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
mistercrunch commented on issue #28259:
URL: https://github.com/apache/superset/issues/28259#issuecomment-2265711907
> the JS Chunk load errors have become so severe
I thought the JS Chunk load errors would be corollated to [how many users
have an active browser open updates] * [how
mistercrunch commented on PR #29798:
URL: https://github.com/apache/superset/pull/29798#issuecomment-2265704387
I just created a new `hold:updating.md` label we can use that should prevent
merging without having to do a "requires changes" review
--
This is an automated message from the Ap
rusackas opened a new pull request, #29838:
URL: https://github.com/apache/superset/pull/29838
### SUMMARY
Just a good ol' `npm audit fix` as we should do periodically. Removes two
`high` level dependencies.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
### TE
Vitor-Avila commented on code in PR #29802:
URL: https://github.com/apache/superset/pull/29802#discussion_r1701959131
##
superset/db_engine_specs/databricks.py:
##
@@ -434,7 +434,25 @@ def get_default_catalog(
cls,
database: Database,
) -> str | None:
+
Vitor-Avila commented on code in PR #29802:
URL: https://github.com/apache/superset/pull/29802#discussion_r1701945638
##
pyproject.toml:
##
@@ -236,7 +236,7 @@ legacy_tox_ini = """
# Remember to start celery workers to run celery tests, e.g.
# celery --app=superset.tasks.celer
sfirke commented on issue #29483:
URL: https://github.com/apache/superset/issues/29483#issuecomment-2265284640
Thank you for sharing what worked for you! Until Superset is tested
continuously to work with MariaDB, I would be worried about people setting up
new instances using it as a backe
rajivpatki commented on issue #29835:
URL: https://github.com/apache/superset/issues/29835#issuecomment-2265239548
Try
```bash
pip install --no-cache-dir s3werkzeugcache
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
michael-s-molina commented on PR #29798:
URL: https://github.com/apache/superset/pull/29798#issuecomment-2265231752
I'm sorry @mistercrunch, I think I missed your comment. I agree that a
comment in `UPDATING.md` is necessary. @danielli-ziprecruiter could you open a
follow-up please?
--
T
philicious commented on issue #28259:
URL: https://github.com/apache/superset/issues/28259#issuecomment-2265205195
Since upgrading to 4.x, the JS Chunk load errors have become so severe that
we had to downgrade again.
(We have several dozen users and use auto-refreshing dashboards alot)
enricozab commented on issue #29825:
URL: https://github.com/apache/superset/issues/29825#issuecomment-2265089931
I have also fixed this issue by increasing my WSL memory. However, I
encounter this issue where I get stuck at "**[webpack.Progress] 94% sealing
after seal**".
Found but failed to import local superset_config
Traceback
dosubot[bot] commented on issue #29835:
URL: https://github.com/apache/superset/issues/29835#issuecomment-2264991382
The error `ModuleNotFoundError: No module named 's3cache'` indicates that
the `s3cache` module is not installed in your environment. The `s3cache` module
is an external d
hieuthanhpro opened a new issue, #29835:
URL: https://github.com/apache/superset/issues/29835
### Bug description
when I use: from s3cache.s3cache import S3Cache
logs is: ModuleNotFoundError: No module named 's3cache'
thank you!
### How to reproduce the bug
1. code:
marre commented on PR #29287:
URL: https://github.com/apache/superset/pull/29287#issuecomment-2264889422
Noticed that I had a typo in the first sentence of the description. I wrote
"modified-by" where I intended "owner" and vice-versa. Corrected in description.
--
This is an automated mes
marre commented on PR #29287:
URL: https://github.com/apache/superset/pull/29287#issuecomment-2264821230
@sadpandajoe Would really love to have this in 4.1...
@bruno-santos-6 What version of superset are you testing with?
Just pulled latest from superset/superset (master branch)
k-e-das commented on issue #24579:
URL: https://github.com/apache/superset/issues/24579#issuecomment-2264775075
We haven't found any Content Security Policy (CSP) defined in the
configurations. Please make sure to configure CSP using the TALISMAN_ENABLED
and TALISMAN_CONFIG keys or any
57 matches
Mail list logo
