Skip to content

Conversation

@koszti
Copy link
Member

@koszti koszti commented Nov 16, 2025

Description

The query details page in the Preview UI fails when catalogProperties are set for a given query session. For example:

{
...
        "systemProperties": {
            "query_max_run_time": "3600000ms"
        },
        "catalogProperties": {
            "data_lake_iceberg": {
                "max_partitions_per_writer": "500",
                "expire_snapshots_min_retention": "2h",
                "remove_orphan_files_min_retention": "2d"
            }
        },
...

The react component does not properly handle nested catalogProperty objects and throws the following exception:

react-dom_client.js?v=2c8e0913:5440 Uncaught Error: Objects are not valid as a React child (found: object with keys {max_partitions_per_writer, expire_snapshots_min_retention, remove_orphan_files_min_retention}). If you meant to render a collection of children, use an array instead.

This PR updates the component to correctly render nested session catalog properties.

Additional context and related issues

After the fix catalog properties are displayed as expected:

image

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(x) Release notes are required, with the following suggested text:

## Section
* Fix preview UI to render catalog properties correctly. ({issue}`27327`)

@cla-bot cla-bot bot added the cla-signed label Nov 16, 2025
@github-actions github-actions bot added the ui Web UI label Nov 16, 2025
@koszti koszti force-pushed the fix-preview-ui-catalog-properties branch from 6a03a7f to 7b0d70f Compare November 16, 2025 07:31
@wendigo wendigo force-pushed the fix-preview-ui-catalog-properties branch from 7b0d70f to 5b47ac3 Compare November 18, 2025 22:51
@wendigo
Copy link
Contributor

wendigo commented Nov 18, 2025

(rebased)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants