OIiveirra commented on PR #69528:
URL: https://github.com/apache/doris/pull/69528#issuecomment-6399562602

   <!-- doris-repo-review:v1:begin -->
   ### Local pipeline review — ✅ PASS
   
   ```yaml
   schema: doris-repo-review/v1
   status: PASS
   pr: apache/doris#69528
   commit: b1fb0a3fe44334d62eacb8590dbae3aa1ac49e72
   base: 00a9e1cb6dd99355f1b385e3c3ec5c23ae5a83cf
   reviewed_at: 2026-10-25T23:31+08:00
   reviewer: OIiveirra
   model: gpt-6.1-sol
   effort: xhigh
   findings: {blocker: 0, major: 0, minor: 0, nit: 0}
   rounds: 1
   converged: true
   ```
   
   **Notes for maintainers**
   
   - 
`fe/be-java-extensions/hadoop-deps/src/main/java/org/apache/hadoop/fs/FileSystem.java:4265`
 — finite same-table ViewFS aliases now use unambiguous resolved table/path 
identity. Actual pinned Hadoop3.4.2 SINGLE lazy initialization and real 
unguarded/guarded dispatch independently establish the known route.
   - 
`fe/be-java-extensions/hadoop-deps/src/main/java/org/apache/hadoop/fs/FileSystem.java:4373`
 — successful state insertion has matching finally removal; repeated states, 
growing-path cycles, local targets and the independent32-state bound remain 
rejected as appropriate.32 mount states may terminate remotely;33 rejects.
   - 
`fe/fe-core/src/test/java/org/apache/doris/tablefunction/HadoopTvfRuntimeAdmissionTest.java:391`
 — new controls cover actual ViewFS stat/literal-delete paths, native 
retained-Context admission, both LOCAL settings and static32/33 boundary. Only 
recognized remote leaf storage is mocked; real native writer IO or32-hop 
network execution is not claimed.
   - Full65-file review converged in round1 with zero findings. Parent 
source-bound129 FE tests/nine fresh required XML and26-file style pass; review 
ran no tests/builds. Unchanged C++/SDK/Thrift evidence is reused by bytes, 
without claiming full CI/SQL/mixed-version execution. Master and4.2 are 
excluded.
   
   <sub>Reviewed locally with the `doris-repo-review` pipeline. Repository 
policy may accept this receipt for the matching commit; it is not a human 
Apache approval.</sub>
   <!-- doris-repo-review:v1:end -->
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to