Github user FlorianHockmann commented on the issue:

    https://github.com/apache/tinkerpop/pull/915
  
    @redoz Currently, the status code is not extended by providers to my 
knowledge. The status codes are documented 
[here](http://tinkerpop.apache.org/docs/current/dev/provider/#_graph_driver_provider_requirements)
  in case you didn't find that already and providers seem to use the very 
general status code `500 - SERVER ERROR` for errors specific to their 
implementation like the `Request Rate to Large` error of Cosmos DB.
    
    I think we should stick with the enum for now. If we want to allow 
providers to add their own status code, then that should be discussed 
separately from this PR. At least, the Java driver would [also need an 
update](https://github.com/apache/tinkerpop/blob/7a814e196519775079005cdf9bc73ec934302891/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/message/ResponseStatusCode.java)
 for that. So, it would be a bigger change than just transforming the C# enum 
into a class with constants.


---

Reply via email to