* bump snowflake for java 17+

* Update modules/drivers/snowflake/deps.edn



* Update deps.edn

---------

Co-authored-by: dpsutton <dan@dpsutton.com>
Co-authored-by: Luis Paolini <paoliniluis@gmail.com>
Co-authored-by: Cam Saul <1455846+camsaul@users.noreply.github.com>
Co-authored-by: Uladzimir Havenchyk <125459446+uladzimirdev@users.noreply.github.com>

Make sure a new table gets table-level block if any other tables have table-level block (#49509) (#49636)

Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>

* Re-introduce "Upload CSV" button for instances with DWH attached (#49545)

* Move the upload CSV button to the main nav bar

* Remove the old `SidebarOnboardingSection`

* Remove obsolete unit tests

* Rework E2E tests

* Add some spacing around the button

* Fix E2E test

* Break `MainNavbar` unit test down

* Expand unit tests

* Update E2E tests

* Fix E2E test failure

* Move JSDoc where it belongs

* Provide context for translators

* Simplify the condition

* Make "Add database" more prominent (#49678)

* Remove dead code from browse models page

* Use the grid to control the gap

* Implement "Add database" button

* Add basic unit tests

* Make sure the feature works when there are no databases

* Add analytics to the db list

* Add "Add database" button to the sidebar

* Add E2E Snowplow tests

* Add unit tests

* Deal with authentication in each beforeEeach separately

* Add Snowplow E2E test for a database card

* Slightly tweak the title line height

* Fix flaky tests

* Fix flaky tests

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

Enable debugUnreturnedConnectionStackTraces in data-warehouse-connection-pool-properties (#49456) (#49807)

* Add blurb from C3P0 docs to comment for unreturnedConnectionTimeout

* Enable debugUnreturnedConnectionStackTraces in data-warehouse-connection-pool-properties

Closes #47981

* Add defsetting for jdbc-data-warehouse-debug-unreturned-connection-stack-traces

And only enable debugUnreturnedConnectionStackTraces in the C3P0 config if the setting is enabled. There is some
memory cost to enabling the setting, and since maxPoolSize is user-configurable, it seems safer to disable
debugUnreturnedConnectionStackTraces by default.

* Add metabase.logger/level-enabled?

* Log a warning if C3P0 logs are not configured correctly to log stack traces

Log a warning if the user enables the jdbc-data-warehouse-debug-unreturned-connection-stack-traces but does not raise
the log level for the com.mchange package to INFO or higher.

* Add docs for MB_JDBC_DATA_WAREHOUSE_DEBUG_UNRETURNED_CONNECTION_STACK_TRACES

* Improve markdown docs for MB_JDBC_DATA_WAREHOUSE_DEBUG_UNRETURNED_CONNECTION_STACK_TRACES



---------

Co-authored-by: appleby <86076+appleby@users.noreply.github.com>
Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Enforce collection perms on native source-card IDs instead of requiring native query perms (#49364) (#49796)

Co-authored-by: Noah Moss <32746338+noahmoss@users.noreply.github.com>

Fixes: #49786

Co-authored-by: Case Nelson <case@metabase.com>

* Keep Pivoted Option works in Subscriptions

Fixes: #49525

Even though the user could toggle 'Keep data pivoted` in the Sharing Sidebar UI, the setting was not being propagated
to the backend, so attachments would not be pivoted.

* add a cypress test

* add viz settings to get the pivot working

* fix for linter

* remove eslint ignore

---------

Co-authored-by: adam-james <21064735+adam-james-v@users.noreply.github.com>
Co-authored-by: Aleksandr Lesnenko <alxnddr@gmail.com>

* Revert "[streaming-response] Use HttpEndpoint.isOpen() instead of reading a byte from TCP socket (#49460)"

This reverts commit 060baf0c.

* Disable connection reuse for requests that serve streaming responses

Co-authored-by: Oleksandr Yakushev <alex@bytopia.org>

* add notes on oracle connection

* edit

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

* docs: update links to learn

* bad replace

* Update docs/exploration-and-organization/events-and-timelines.md



---------

Co-authored-by: Alex Yarosh <alexandra@metabase.com>
Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

fix(sdk): generated cli components has inconsistent styles after the style leak fix (#49723) (#49775)

Co-authored-by: Phoomparin Mano <poom@metabase.com>

* Ensure that correct error messages show up in the Edit user modal (#48966)

* kick ci

---------

Co-authored-by: Raphael Krut-Landau <raphael.kl@gmail.com>

* Give the command palette a higher z-index (#49387)

* kick ci

---------

Co-authored-by: Raphael Krut-Landau <raphael.kl@gmail.com>

* Correct embedding hash options type semantically

* Allow smart scalar text to be translated

* Stop using global variable

* Add more details to embedding types

* Fix public embed not working with `?locale` anymore

* Doesn't pollute SDK user window object

Co-authored-by: Oisin Coveney <oisin@metabase.com>

* Remove @flaky tag

* Unskip test and remove @flaky tag

* Remove @flaky tag

* Remove @flaky tag

* Revert "Remove @flaky tag"

This reverts commit 08c57add.

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

Co-authored-by: Phoomparin Mano <poom@metabase.com>

* Try to fix flake

* See if timeout is necessary

* Revert "See if timeout is necessary"

This reverts commit 181f3c5e.

* Revert change for coordinates filter

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

* Remove @flaky tag

* Make workaround test pass

* Disable workaround test

* Uncomment and unquarantine the workaround test

* Disable normal test for now

* Revert "Disable normal test for now"

This reverts commit 68fa2175.

* Bring back quarantine

* Update comment

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

Co-authored-by: Phoomparin Mano <poom@metabase.com>

* Handle touch start capture as well as mouse down for popover

* Also stop event propagation for touch events on menu item

Co-authored-by: Romeo Van Snick <romeo@romeovansnick.be>

Co-authored-by: Alexander Solovyov <alexander@solovyov.net>

* notes on database browser

* typo

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

* docs: cloud environment variables

* Apply suggestions from code review



---------

Co-authored-by: Alex Yarosh <alexandra@metabase.com>
Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Nicolò Pretto <info@npretto.com>

Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

Co-authored-by: Oisin Coveney <oisin@metabase.com>

* note about apsaradb

* Update docs/installation-and-operation/configuring-application-database.md



---------

Co-authored-by: Luis Paolini <paoliniluis@gmail.com>
Co-authored-by: Jeff Bruemmer <jeff.bruemmer@gmail.com>

* docs: remove nested queries setting

* docs: remove mention of nested query setting

Co-authored-by: Alex Yarosh <alexandra@metabase.com>

Co-authored-by: Ngoc Khuat <qn.khuat@gmail.com>

* move the database prompt to the very beginning

* ability to use sample database for cli

* print username and password in console

* no tenancy isolation as a default option

* highlight sample credentials file

* update credential log messages



* continue without tenancy isolation

---------

Co-authored-by: Phoomparin Mano <poom@metabase.com>
Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

Fix - Values from the returned rows not present in the visualization are interpolable in the URL but don't have values (#49530) (#49654)

* Fix #46318

* Add repro for #46318

* Move setup to beforeEach

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

* Remove @flaky tag

* Fix flake

* Try typing delay instead

* Improve comment

* Bring back original solution

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

* Pivot Measures Order Used in Pivot Exports

Fixes #48442

A pivot table can have any number of measures, and the user can order these by dragging in the UI. Before this PR,
that order was ignored and measures would alway be in index order, which is confusing for any user who needs the
measures to be displayed in a particular order, especially if they've re-ordered them in the pivot viz settings UI.

A test has been added to check that measure order is used.

A few minor changes to the pivot qp and post-processor
 - measure indices are looked up in the pivot qp and added in viz-settings order
 - the pivot measures are only added if the qp has not already added them.
 - pivot-opts Malli spec has been made in the namespace, adjusted to allow `nil` as a valid pivot-opts output, and
 used in relevant functions

* address review points.

* add a rows order test

* Filter out Empty Rows from Pivot Exports

Fixes #49353

The linked issue is not actually related to pivot export size but is instead related to the 'Min of Created At: Month'
aggregation; the default aggregation function was `+`, so it broke when the date string was encountered.

That was fixed.

As I was trynig to keep the export small if possible, I noticed that in some cases empty rows are appended, so I added
the filter so that if a pivot row's values are completely empty, it doesn't add it.

Finally, this PR also adds the 'sub section totals' which I noticed were missing from the exports. This comes up when
you have 3+ pivot-rows configured, so you can see the subtotals for the first pivot row and the subtotals nested
within those sections for the second pivot row, and so on.

* add test for non-numeric values

* Make sure the new name refs match on aggregations not just breakouts

* cljfmt

Co-authored-by: adam-james <21064735+adam-james-v@users.noreply.github.com>

previously import would return 200 w/o regard if it was successful or not

fixes #49602

Co-authored-by: Alexander Solovyov <alexander@solovyov.net>

* Fix :pivot-measures with column names

* Fix the test

* Update src/metabase/query_processor/pivot.clj



* Add a test

* Add a test

---------

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>
Co-authored-by: Braden Shepherdson <braden@metabase.com>

* Remove @flaky tag

* Fix flake

* Refactor

* Use getDashboardCard

* Shorten text

* Fix flake

* Revert previous changes

* Revert "Revert previous changes"

This reverts commit 16b56f26.

* Fix flake

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>