jerryshao commented on code in PR #13067:
URL: https://github.com/apache/gravitino/pull/13067#discussion_r3976550830


##########
docs/iceberg-rest-service.md:
##########
@@ -724,7 +724,7 @@ The Iceberg REST server exposes three health check 
endpoints following the same
 
 | Endpoint                    | Description                                    
                                                                            | 
HTTP status |
 
|-----------------------------|----------------------------------------------------------------------------------------------------------------------------|-------------|
-| `GET /iceberg/health/live`  | Liveness probe. Returns 200 as long as the 
HTTP server thread can respond.                                                 
| 200         |
+| `GET /iceberg/health/live`  | Liveness probe. Returns 200 when the HTTP 
thread can respond and no OOM has been observed.                                
 | 200 / 503   |

Review Comment:
   You should mention when to return 503 in the Description.



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

Reply via email to