Skip to content

Commit

Permalink
Upgrade Thrift compiler to v0.19 and regenerate types (jaegertracing#…
Browse files Browse the repository at this point in the history
…5080)

## Which problem is this PR solving?
- Use latest version of Thrift compiler

## Description of the changes
- Upgrade to v0.19 Docker image of Thrift compiler
- Regenerate types

## How was this change tested?
- CI

Signed-off-by: Yuri Shkuro <[email protected]>
  • Loading branch information
yurishkuro authored Jan 4, 2024
1 parent a7c5ef0 commit 42304f3
Show file tree
Hide file tree
Showing 16 changed files with 366 additions and 177 deletions.
2 changes: 1 addition & 1 deletion Makefile.Thrift.mk
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Copyright (c) 2023 The Jaeger Authors.
# SPDX-License-Identifier: Apache-2.0

THRIFT_VER=0.14
THRIFT_VER=0.19
THRIFT_IMG=jaegertracing/thrift:$(THRIFT_VER)
THRIFT=docker run --rm -u ${shell id -u} -v "${PWD}:/data" $(THRIFT_IMG) thrift
THRIFT_GO_ARGS=thrift_import="github.com/apache/thrift/lib/go/thrift"
Expand Down
2 changes: 1 addition & 1 deletion thrift-gen/agent/GoUnusedProtection__.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 10 additions & 3 deletions thrift-gen/agent/agent-consts.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

31 changes: 18 additions & 13 deletions thrift-gen/agent/agent.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion thrift-gen/baggage/GoUnusedProtection__.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 10 additions & 3 deletions thrift-gen/baggage/baggage-consts.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

92 changes: 59 additions & 33 deletions thrift-gen/baggage/baggage.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion thrift-gen/jaeger/GoUnusedProtection__.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 10 additions & 3 deletions thrift-gen/jaeger/jaeger-consts.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 42304f3

Please sign in to comment.