raghav-reglobe commented on PR #70349:
URL: https://github.com/apache/doris/pull/70349#issuecomment-6682249927

   <!-- doris-repo-review:v1:begin -->
   ### Local pipeline review — ✅ PASS
   
   ```yaml
   schema: doris-repo-review/v1
   status: PASS
   pr: apache/doris#70349
   commit: 5158d9ffce8fecab6e6088656d6792b840f50ca3
   base: 4c63661f60f96c694757000640bcebc832e8bb6d
   reviewed_at: 2026-11-06T15:31+05:30
   reviewer: raghav-reglobe
   model: claude-fable-5-1
   effort: xhigh
   findings: {blocker: 0, major: 0, minor: 0, nit: 3}
   rounds: 1
   converged: true
   ```
   
   **Notes for maintainers**
   
   - 
`fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/Auth.java:2324-2332` 
— the row is sized from `MysqlDBTable`'s schema and the FE asserts that the 
table ends with exactly `PasswordPolicy.MYSQL_USER_COLUMNS`; the BE's 
`SchemaUserScanner` list (the third copy, positional) is pinned by the new 
`auth_p0` suite rather than by that assertion.
   - `be/src/information_schema/schema_user_scanner.cpp:65` — mixed versions: a 
new BE fills the column empty from an old FE's rows; an old BE cannot serve the 
new column to a new FE (`SELECT *` from `mysql.user` fails with `no match 
column` until that BE is upgraded), as for any schema-table column.
   - Every expected value in `AccountLockTest` and the suite was re-derived 
from the code (`2592000` = 30 days, `86400` = 1 day, `""` for an unset lock 
time, `***` for the masked history).
   
   <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