Commit Graph

5 Commits

Author SHA1 Message Date
Pranjal Raihan
bf1c11ffdd Remove apache::thrift::Client<Foo>::header_ methods by default
Summary:
Re-do of D60793177 with the `fb303_core.thrift`-related stuff fixed. This prevents build failures like:
```
$ buck2 build servicefoundry/experimental/marios:thrift-ast
Action failed: fbcode//servicefoundry/experimental/marios:thrift-ast-thrift/qos_basecontroller_test.thrift (genrule_thrift)
Local command returned non-zero exit code 1
Reproduce locally: `env -- 'ASAN_OPTIONS=detect_leaks=0,detect_odr_violation=0' 'BUCK_SCRATCH_PATH=buck-out/v2/tmp/fbcod ...<omitted>... ervicefoundry/experimental/marios/__thrift-ast-thrift/qos_basecontroller_test.thrift__/sh/genrule.sh (run `buck2 log what-failed` to get the full command)`
stdout:
stderr:
ln: ./thrift: cannot overwrite directory
```
Which is caused by Thrift files being in a different directory than their owning `TARGETS` file.

This enforces the contract outlined in {D60782227}. In short, we're going to stop generating `AsyncClient::header_*` variants of methods. We've already added annotations for existing code that use the `header_*` methods. As long as the builds don't break, this is deleting unused code. This should help us reclaim some binary size headroom so we can implement `ClientInterceptor`s.

Reviewed By: sazonovkirill

Differential Revision: D61552181

fbshipit-source-id: 1b460dfb425aa93afcb90cefdaae8def5abb866c
2024-08-29 13:18:27 -07:00
Tejas Patil
d161ebcb7b Revert D60793177: Remove apache::thrift::Client<Foo>::header_ methods by default
Differential Revision:
D60793177

Original commit changeset: c02254485b4f

Original Phabricator Diff: D60793177

fbshipit-source-id: d22b14db6a80910f401fcbafedfb7dafaf9a598f
2024-08-20 06:27:29 -07:00
Pranjal Raihan
b9d706e3d4 Remove apache::thrift::Client<Foo>::header_ methods by default
Summary: This enforces the contract outlined in {D60782227}. In short, we're going to stop generating `AsyncClient::header_*` variants of methods. We've already added annotations for existing code that use the `header_*` methods. As long as the builds don't break, this is deleting unused code. This should help us reclaim some binary size headroom so we can implement `ClientInterceptor`s.

Reviewed By: nickbrekhus

Differential Revision: D60793177

fbshipit-source-id: c02254485b4f22b5215a2b26453f9365fb4f7829
2024-08-19 19:33:26 -07:00
TJ Yin
d9aeb7dc90 Add package keyword
Reviewed By: avalonalex

Differential Revision: D58901118

fbshipit-source-id: 2b33506542a8a7fe02d81e030ed1a000fbd273f7
2024-06-24 09:47:15 -07:00
Facebook GitHub Bot
1669ef5b5a Re-sync with internal repository
The internal and external repositories are out of sync. This Pull Request attempts to brings them back in sync by patching the GitHub repository. Please carefully review this patch. You must disable ShipIt for your project in order to merge this pull request. DO NOT IMPORT this pull request. Instead, merge it directly on GitHub using the MERGE BUTTON. Re-enable ShipIt after merging.
2024-02-21 21:16:37 -05:00