Skip to content

[test] Fix CsvSkipEmptyLinesTableTest broken by deferred file-store validation - #10989

Merged
JingsongLi merged 1 commit into
apache:masterfrom
cxzl25:fix-csv-skip-empty-lines-test
Nov 5, 2026
Merged

JingsongLi merged 1 commit into
apache:masterfrom
cxzl25:fix-csv-skip-empty-lines-test

Conversation

@cxzl25

@cxzl25 cxzl25 commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Purpose

https://github.com/apache/paimon/actions/runs/41786690988/job/126452438400

[ERROR] org.apache.paimon.table.CsvSkipEmptyLinesTableTest.testRejectedAsDynamicOptionOfFileStoreTable  Time elapsed: 0.127 s  <<< FAILURE!
java.lang.AssertionError: 

Expecting code to raise a throwable.
	at org.apache.paimon.table.CsvSkipEmptyLinesTableTest.testRejectedAsDynamicOptionOfFileStoreTable(CsvSkipEmptyLinesTableTest.java:95)

Tests

GHA

sundapeng added a commit to sundapeng/paimon that referenced this pull request Nov 5, 2026
Discover built-in index factories with the interface classloader so a
blind JobMaster context classloader cannot initialize an empty registry.
On a lookup miss, discover optional providers with the current context
classloader without retaining them in the static registry.

Cover child-only provider discovery after blind initialization, sibling
classloader isolation, registration checks, and global file selection.
Align the CSV dynamic-option test with deferred runtime validation;
the test-contract adjustment follows apache#10989.
@JingsongLi
JingsongLi merged commit 90201ee into apache:master Nov 5, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants