mchades

Results 76 comments of mchades

Hi, @mygrsun could you plz resolve the comment and CI issue? Thanks

I have verified locally that through this PR, the time consumption of listing 1000 tables can be reduced from **2043ms** to **14ms**. It's ready for review now. @jerryshao

How about using the testConnection endpoint: https://gravitino.apache.org/docs/0.8.0-incubating/api/rest/test-connection

> 你好,我看到在 web/web/src/lib/utils/regex.js 中这里对name的输入做了正则限制,这里做修改应该可以解决,请问这个可以分配给我作为我的第一个任务吗? You should update the regular expression here: https://github.com/apache/gravitino/blob/6da2f79fe2d69e3d32ca372c4ae407ab66002872/core/src/main/java/org/apache/gravitino/catalog/CatalogNormalizeDispatcher.java#L51

> [@mchades](https://github.com/mchades) Should we include this fix in branch 0.9? yes, I think so.

All comments resolved, plz help to review again when you have time, thanks! @jerryshao

@koonchen No problem! Please go ahead and feel free to ping me if you have any questions

@AbhiSharmaNIT Yeah, you summed it up correctly! I have not seen @koonchen submit the relevant PR so far, so I think if you are interested in this issue, you can...