Skip to content

Conversation

@xiaochen-zhou
Copy link
Contributor

@xiaochen-zhou xiaochen-zhou commented Aug 4, 2025

Purpose of this pull request

There might be some ambiguity in the documentation. When implementing multi-table reading in StarRocks, the schema has been moved to table_list, and the option rule validation has also been changed to optional. Therefore, updated the corresponding documentation to indicate that the schema is now marked as 'optional'. However, in practice, whether it is table or table_list, the schema is still required.

close: 9647

source {
  StarRocks {
    nodeUrls = ["starrocks_e2e:8030"]
    username = root
    password = ""
    database = "test"
    table_list = [
    {
        table = "e2e_table_source"
        schema = {
            fields {
               BIGINT_COL = BIGINT
               LARGEINT_COL = STRING
               SMALLINT_COL = SMALLINT
               TINYINT_COL = TINYINT
               BOOLEAN_COL = BOOLEAN
               DECIMAL_COL = "DECIMAL(20, 1)"
               DOUBLE_COL = DOUBLE
               FLOAT_COL = FLOAT
               INT_COL = INT
               CHAR_COL = STRING
               VARCHAR_11_COL = STRING
               STRING_COL = STRING
               DATETIME_COL = TIMESTAMP
               DATE_COL = DATE
            }
        }
    },
    {
        table = "e2e_table_source_2"
        schema = {
            fields {
            }
        }
    }]
    scan_batch_rows = 10
    max_retries = 3
    scan.params.scanner_thread_pool_thread_num = "3"
    
  }
}

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@Hisoka-X Hisoka-X changed the title [Fix][Connector-V2] Update StarRocks doc: Change schema necessity to true [Fix][Doc] Update StarRocks doc change schema necessity to true Aug 4, 2025
@corgy-w corgy-w merged commit 45f8ac6 into apache:dev Aug 4, 2025
5 checks passed
@xiaochen-zhou xiaochen-zhou deleted the fix-sr-doc branch October 9, 2025 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

为什么2.3.11seatunnel 的source StarRocks还需要显示配置schema,但是官方文档显示不需要显示配置

3 participants