[GitHub] [thrift] Jimexist merged pull request #2686: THRIFT-5644: java to use jdk 17 and upgrade minimal language level to 11

2022-10-13 Thread GitBox
Jimexist merged PR #2686: URL: https://github.com/apache/thrift/pull/2686 -- 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:

[jira] [Commented] (THRIFT-5587) Introduce uuid as additional builtin type

2022-10-13 Thread Yuxuan Wang (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17617194#comment-17617194 ] Yuxuan Wang commented on THRIFT-5587: - Actually there's option 3, which might be the easiest to

Re: Context around TType in Java

2022-10-13 Thread Yuxuan Wang
My guess would be that ENUM is just a wrong TType that's not really used in the code. Over the wire we just write/read enums as i32s. And you might have already noticed that it's not in the spec. On Thu, Oct 13, 2022 at 1:19 AM Jiayu Liu wrote: > Hi thrift-dev, > > I would like to know more

[jira] [Commented] (THRIFT-5587) Introduce uuid as additional builtin type

2022-10-13 Thread Yuxuan Wang (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17617175#comment-17617175 ] Yuxuan Wang commented on THRIFT-5587: - [~jensg] in the example this is a valid UUId string (in the

[jira] [Updated] (THRIFT-5650) Add UUID support in go

2022-10-13 Thread Liu Jiayu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5650?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Liu Jiayu updated THRIFT-5650: -- Summary: Add UUID support in go (was: add uuid support in go) > Add UUID support in go >

[jira] [Updated] (THRIFT-5650) add uuid support in go

2022-10-13 Thread Liu Jiayu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5650?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Liu Jiayu updated THRIFT-5650: -- Parent: THRIFT-5587 Issue Type: Sub-task (was: Improvement) > add uuid support in go >

[jira] [Updated] (THRIFT-5629) Add UUID support for Swift

2022-10-13 Thread Liu Jiayu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5629?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Liu Jiayu updated THRIFT-5629: -- Parent: THRIFT-5587 Issue Type: Sub-task (was: Improvement) > Add UUID support for Swift >

[jira] [Updated] (THRIFT-5648) Add UUID support for rust

2022-10-13 Thread Liu Jiayu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5648?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Liu Jiayu updated THRIFT-5648: -- Parent: THRIFT-5587 Issue Type: Sub-task (was: Improvement) > Add UUID support for rust >

[GitHub] [thrift] dependabot[bot] opened a new pull request, #2712: Bump json-schema and jsprim in /lib/ts

2022-10-13 Thread GitBox
dependabot[bot] opened a new pull request, #2712: URL: https://github.com/apache/thrift/pull/2712 Bumps [json-schema](https://github.com/kriszyp/json-schema) and [jsprim](https://github.com/joyent/node-jsprim). These dependencies needed to be updated together. Updates `json-schema` from

[GitHub] [thrift] dependabot[bot] opened a new pull request, #2711: Bump json-schema and jsprim

2022-10-13 Thread GitBox
dependabot[bot] opened a new pull request, #2711: URL: https://github.com/apache/thrift/pull/2711 Bumps [json-schema](https://github.com/kriszyp/json-schema) and [jsprim](https://github.com/joyent/node-jsprim). These dependencies needed to be updated together. Updates `json-schema` from

[GitHub] [thrift] dependabot[bot] closed pull request #2707: Bump markdown-it from 10.0.0 to 12.3.2 in /lib/js

2022-10-13 Thread GitBox
dependabot[bot] closed pull request #2707: Bump markdown-it from 10.0.0 to 12.3.2 in /lib/js URL: https://github.com/apache/thrift/pull/2707 -- 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

[GitHub] [thrift] Jens-G merged pull request #2706: Bump marked from 2.0.7 to 4.1.1 in /lib/js

2022-10-13 Thread GitBox
Jens-G merged PR #2706: URL: https://github.com/apache/thrift/pull/2706 -- 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:

Context around TType in Java

2022-10-13 Thread Jiayu Liu
Hi thrift-dev, I would like to know more about the context of TType in Java, specifically when ENUM was introduced in [1] in 2009, it is placed with value 16 (binary protocol) but in swift [2] it is utf8, in delphi [3] uuid, in C# [4] uuid, and per spec [5] uuid. Using 16 for uuid will possibly

[jira] [Commented] (THRIFT-5587) Introduce uuid as additional builtin type

2022-10-13 Thread Jens Geyer (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17616791#comment-17616791 ] Jens Geyer commented on THRIFT-5587: See ConstantsDemo.thrift there is a test case for it.