The GitHub Actions job "E2E - Agent" on 
shardingsphere.git/fix/oracle-timestamp-with-timezone-order-by has failed.
Run started by GitHub user zhan7236 (triggered by zhan7236).

Head commit for run:
684870ce53f1085545b32f8406aa460a03899f21 / zhan7236 
<[email protected]>
refactor: rename getTimestampValue to getDefaultValue and fix default behavior

Address review feedback from maintainer @terrymanu:
- Renamed getTimestampValue() to getDefaultValue() to better reflect its purpose
- Changed default implementation from getTimestamp() to getObject() to preserve
  backward compatibility for non-timestamp types (JSON, UUID, BIT, etc.)
- Oracle's vendor-specific type codes (-101/-102) are now handled solely inside
  OracleResultSetMapper, preventing unintended side effects for other dialects

This ensures:
1. Only Oracle TIMESTAMP WITH TIME ZONE types are specially handled
2. All other unhandled types continue to use getObject() as before
3. No compatibility issues for MySQL/PostgreSQL/openGauss dialects

Report URL: https://github.com/apache/shardingsphere/actions/runs/19797282522

With regards,
GitHub Actions via GitBox

Reply via email to