CRZbulabula opened a new pull request, #18293:
URL: https://github.com/apache/iotdb/pull/18293

   ## Description
   
   ### Error message
   
   Replace the misleading `read-only` error used for unsupported operations on 
the audit database with a more accurate message:
   
   > Apache IoTDB does not support this operation on database '%s'.
   
   The message is shared by the tree-model and table-model authorization paths 
and is localized in both English and Chinese.
   
   ### Affected operations
   
   - Apply the new message to tree-model write, schema, template, TTL, and 
database deletion checks that previously reported the audit database as 
read-only.
   - Apply the same message to unsupported table-model operations on `__audit`.
   - Keep the existing reserved-name, grant/revoke, and AUDIT-admin query 
messages unchanged because they already describe their restrictions precisely.
   
   ### Test coverage
   
   - Add coverage for `DELETE DATABASE root.__audit` and template operations in 
`IoTDBTemplateAuthIT`.
   - Update TTL coverage in `IoTDBSeriesPermissionIT`.
   - Add coverage for `DROP DATABASE __audit` in `IoTDBAuthenticationTableIT`.
   
   Verified with:
   
   - Full-reactor English and Chinese test compilation.
   - `IoTDBTemplateAuthIT#manageDataBaseTest`.
   - `IoTDBSeriesPermissionIT#ttlOperationsTest`.
   - `IoTDBAuthenticationTableIT#testInsert` with the table-simple profile.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [x] added comments explaining the intent where it was not obvious.
   - [x] added or updated integration tests.
   - [x] been tested in a test IoTDB cluster.
   
   <hr>
   
   ##### Key changed/added classes (or packages if there are too many classes) 
in this PR
   
   - `AccessControlImpl`
   - `ITableAuthCheckerImpl`
   - `TreeAccessCheckVisitor`
   - `DataNodeQueryMessages`
   


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