Congxian Qiu
Congxian Qiu
@majin1102 sorry for the late reply, I've drafted a version such that: TableRuntimeMeta is used for retrieving from db, `TableRuntime` is used for data transfer object in ams. could you...
@majin1102 @zhoujinsong @baiyangtx I've updated a version, could you please take a look at this when you're free, thanks
@baiyangtx thanks very much for the review, will update the PR % the search pattern about the db soon, that change will change after we reach the consensus.
@zhoujinsong thanks very much for the review, will update it and attach some result before and after apply the change
sorry for the late reply, will rebase the latest master and attach the result asap.
Finally, got the compare result before and after applying the change. Short answer: This modification has a particularly good performance improvement。 Before the change, the page can be loaded in...
@zhoujinsong @majin1102 thanks for the review and merging!
@tanmaytapan thanks for the reply, maybe you can 1. fork this repo 2. checkout from the main branch in your forked branch named `branch-for-chartjs`(just for easily ref in the below,...
The fix in #3065 also improve this problem(reduce the call to external meta store from two to one), will see is there anything we can do for this rest api
The steps currently implemented are all needed will close this issue now. Please reopen it if there is anything we can do to improve this Rest API.