davidzollo commented on PR #11493:
URL: https://github.com/apache/seatunnel/pull/11493#issuecomment-6703604812

   <!-- daniel-pr-followup-20261107 -->
   Yes, contributing to Apache SeaTunnel.
   
   Pushed `b3aa0f2fefaa3d8164378e983ec85ea1274e91c9` by normal fast-forward 
push. Reconciled dev conflicts while retaining JSON and numeric-special-value 
source options. Added raw JSON support to scalar/array text formats. Fixed 
Doris-specific Stream Load transport so boolean/string scalars and JSON null 
retain their native types; strict parsing remains. The two LocalFile JSON-type 
fixtures now preserve their pre-created writable shared directories instead of 
recreating them as the driver user. Exact existing E2E value/row assertions are 
retained, with new serializer/text regression tests and bilingual migration 
guidance. The follow-up keeps the Doris serializer's original fields and method 
signatures, preserving its implicit Java serial form.
   
   Root cause: the LocalFile logs report Permission denied at the final file 
commit after the schema save mode recreates a previously writable directory. 
Doris 2.1's JSON reader converts top-level boolean tokens to 1/0 and JSON null 
to SQL NULL before the column cast; document-string transport preserves the 
native JSON document. See [Doris 2.1 
reader](https://github.com/apache/doris/blob/2.1.0-rc03/be/src/vec/exec/format/json/new_json_reader.cpp).
 Subsequent Doris checksum failures followed the initial assertion before 
fixture cleanup; they still require the new-head rerun, rather than being 
counted as passed.
   
   Validation: affected-module Spotless and diff whitespace checks passed; one 
independent maintainer source review completed for these deltas. No local 
compilation, unit/IT/E2E execution, Docker or service startup was run under the 
repository validation policy. The [current-head fork 
Build](https://github.com/DanielLeens/seatunnel/actions/runs/42348741989) is 
pending and no runtime pass is claimed. Earlier verification statements refer 
to previous heads. The separate Metalake startup timeout, complete current-head 
CI and real Snowflake-account acceptance remain gates. This PR remains Draft.
   
   The approved merge's POM files exactly match upstream dev; no dependency 
edits were added. Existing connector registrations, options, exact JSON tree 
comparisons and 1-row LocalFile assertions remain.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to