Skip to content

Setting parquet.thrift.string.size.limit = -1 incorrectly throws #3587

@divjotarora

Description

@divjotarora

Describe the bug, including details regarding any error messages, version, and platform.

#3359 added a parquet.thrift.string.size.limit configuration to allow specifying a custom max Thrift message size. If unspecified or explicitly set to -1, the default (100 MB) is used. The config validation code introduced a minor bug here: if the value is explicitly set to -1, maxMessageSize is correctly set to the default, but the configuredMaxMessageSize <= 0 branch will still throw an exception.

Component(s)

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions