[GitHub] [thrift] BytedanceRPC commented on pull request #2469: THRIFT-5423: Support go parameter validation in IDL

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

BytedanceRPC commented on pull request #2469:
URL: https://github.com/apache/thrift/pull/2469#issuecomment-994531708


   > > vt.in: must be in specified values
   > > vt.not_in: must not be in specified values
   > 
   > it's not clear to me whether that is an enumeration (e.g. `vt.in = "[1, 2, 4, 8]"` or a range (e.g. `vt.in = "[1, 4]"`), and if it's a range, are ends inclusive or exclusive?
   > 
   > I think it would help if you can include the markdown file you attached to the JIRA ticket into this PR so I can review the spec first (and it will be easier to give inline feedback to the spec there), before I review the implementation code.
   
   Sorry for not getting back to you sooner. Value of 'vt.in' must be specified，it works like below
   ```thrift
   3: i8 Byte0 = 1 (vt.in = "0", vt.in = "1", vt.in = "2", vt.not_in = "3", vt.not_in = "4", vt.not_in = "5")
   ```
   We update thrift-parameter-validation-proposal, and put it in [doc/proposal/thrift-parameter-validation-proposal.md](https://github.com/apache/thrift/pull/2469/files?file-filters%5B%5D=.md). Thanks for your advice, and welcome to comment.


-- 
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: notifications-unsubscribe@thrift.apache.org

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