Skip to content

Commit 5fa35ae

Browse files
authored
Migrate sampling.thrift (#132)
## Which problem is this PR solving? - jaegertracing/jaeger#6617 ## Description of the changes - generate sampling.thrift code ## How was this change tested? - ci ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `npm run lint` and `npm run test` Signed-off-by: nabil salah <[email protected]>
1 parent 68c7c18 commit 5fa35ae

File tree

4 files changed

+1400
-0
lines changed

4 files changed

+1400
-0
lines changed

Makefile

+1
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ thrift:
5151
$(THRIFT) -o /data --gen go:$(THRIFT_GO_ARGS) -out /data/$(THRIFT_GEN_DIR) /data/thrift/jaeger.thrift
5252
$(THRIFT) -o /data --gen go:$(THRIFT_GO_ARGS) -out /data/$(THRIFT_GEN_DIR) /data/thrift/zipkincore.thrift
5353
$(THRIFT) -o /data --gen go:$(THRIFT_GO_ARGS) -out /data/$(THRIFT_GEN_DIR) /data/thrift/agent.thrift
54+
$(THRIFT) -o /data --gen go:$(THRIFT_GO_ARGS) --out /data/$(THRIFT_GEN_DIR) /data/thrift/sampling.thrift
5455
$(SED) -i.bak 's|"zipkincore"|"$(JAEGER_IMPORT_PATH)/thrift-gen/zipkincore"|g' $(THRIFT_GEN_DIR)/agent/*.go
5556
$(SED) -i.bak 's|"jaeger"|"$(JAEGER_IMPORT_PATH)/thrift-gen/jaeger"|g' $(THRIFT_GEN_DIR)/agent/*.go
5657
rm -rf thrift-gen/*/*-remote thrift-gen/*/*.bak

thrift-gen/sampling/GoUnusedProtection__.go

+6
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

thrift-gen/sampling/sampling-consts.go

+30
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)