tison
After a quick review on the patch, as long as we use `global_reference`, the parallelism is limited to a constant upper bound: > JNI ERROR (app bug): global reference table...
> This limitation is enough for most scenarios If each call obtains one global ref, I'm afraid that the iops is limited by 65535.
@brenobaroni could you provide the environment info? * OS version * Docker provider and version I can build logto image on * OS version - macOS 13.4 * Docker provider...
... while change source to `SELECT \nbar` gives a proper formatted: ``` | 0 | / SELECT 1 | bar | | ^^^ unexpected token | |___^ while parsing statement...
Change `line_start` to `1` also fix the issue. Perhaps it's a bug for `line_start == 0`.
No. The problem is not about `\r\n`, it's about `\t`.
Thanks for your review @briansmith! Do you think it's better to exclude changes around `src/constant_time.rs` in this PR and move forward, or we can wait for https://github.com/briansmith/ring/pull/1899 merged?
Updated. PTAL @briansmith.
@briansmith ping as a reminder :D