[GitHub] [incubator-doris] morningman commented on pull request #4268: [Feature][Web] Add new feature to list all tablets on a particular BE

[GitBox <gi...@apache.org>]

morningman commented on pull request #4268:
URL: https://github.com/apache/incubator-doris/pull/4268#issuecomment-670282812


   That would be a useful feature! But I have some suggestions:
   
   1. There may be hundreds of thousands of tablet on one BE, so it better to add some query parameter like `limit` or `offset` to limit the number of tablets returned. And use a default value, like limit 1000.
   
   2. I sorting out the RESTful API of Doris, in this PR #4261, it only describe the  RESTful API response body of FE, bug I thinks BE can also follow the rule. the response body is like:
   
   ```
       {
        	"msg": "OK",
        	"code": 0,
        	"data": {
        		your real result data....
        	},
        	"count": 0
        }
   ```
   
   3. Document. Every new API need a document in both English and Chinese. You can put them under `administrator-guide/http-actions/`


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org