ClaireLytt commented on PR #39254:
URL: https://github.com/apache/shardingsphere/pull/39254#issuecomment-5102629413
Hi @terrymanu, a quick note on the E2E scenario choice and the fix.
**Why not `tbl`?**
I tried to follow the suggested `MySQL + JDBC + tbl +
HintManager.setDataSourceName("tbl")` setup, but could not reproduce #28841
there. The `tbl` fixture only has sharded tables (`t_order_0` … `t_order_9`),
not a physical `t_order`, so `setDataSourceName("tbl")` with `FROM t_order`
fails with `Table 'tbl.t_order' doesn't exist`. Using `t_order_0` avoids that
error but does not hit the derived-projection / column-label path reliably.
The regression is reproducible with **`readwrite_splitting`**:
`setDataSourceName("write_ds")` + `SELECT T.*, T.status status_new FROM t_order
T WHERE T.order_id = 1000`, which matches the issue pattern (hint to a physical
data source + `T.*` with an extra alias). The focused E2E is scoped to **MySQL
+ JDBC + readwrite_splitting + LITERAL** via `HintManagerColumnLabelE2EIT`.
Regarding https://github.com/apache/shardingsphere/pull/39131 — please give
me a bit more time to go through the latest review comments. Thank you for your
patience!
--
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]
