terrymanu commented on issue #31504: URL: https://github.com/apache/shardingsphere/issues/31504#issuecomment-5327343021
Hi @SankeerthNara, thanks for picking this up — the issue is yours. For anyone checking the current status: this issue should stay open. Of the linked PRs, only #37552 was merged, and it covers the already checked-off `ALTER SYSTEM` item. #38660 attempted the `RECOVER` syntax but was closed without merging, because `RECOVER PARTITION ... AS <new_partition_name>` support and the matching parser IT coverage were still missing. Still to be done on current master: - All `RECOVER DATABASE` / `RECOVER TABLE` / `RECOVER PARTITION` variants listed in the issue; none of them are parseable today (only the unrelated `XA RECOVER` exists in the Doris grammar) - `Array<T> array_cum_sum(Array<T>)` Please follow the procedure in the issue description: verify the syntax against the Doris official docs and a real Doris instance first, then add the grammar rules, visitor logic, and parser IT cases. Submitting one PR per syntax group is preferred. Thanks! -- 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]
