| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
commit_hash:50f729d3716d8b1f5b852cfc008e228172fb79c4
|
|
|
|
| |
commit_hash:87c6f9898873ec96e4aa9dc4c7df86e67a11e313
|
|
|
|
| |
commit_hash:7dbe58a7a8a7980241f20a2e53ace94a62c541df
|
|
|
|
| |
commit_hash:97159680fa733af4212c094e917bc78a47eb0c15
|
|
|
|
| |
commit_hash:cbf5ab65e5ba1b2d286f974a6b6b463ed152b381
|
|
|
|
| |
commit_hash:2404d2a79d4961dedf17f7d0cbec38bcabb78014
|
|
|
|
| |
commit_hash:6f0d83406a245425d156e961c3519d5bd822857a
|
|
|
|
| |
commit_hash:c0112c0409a0972eb859b32a014980020df7ce0a
|
|
|
|
| |
commit_hash:fac37a51fb5b0fdd660beccb43a2a8861308e39f
|
|
|
|
| |
commit_hash:320ab736dea5dae74e2ec8c344fb915be4c4df99
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
TErrorAttribute from yt/core
Plan:
1) Remove `IAttributedDictionary` type from the public API. \+
2) Remove `Set` method from public API in favor of `operator<<=`. \+
3) Adopt `ConvertTo<T>` (or other name) CPO with proper extension into `NYT::NYson::ConvertTo` from `yt/core`.
4) Use CPO from (3) to eliminate direct dependency on `yt/core` of `Get/Find` methods from attributes API.
5) Adopt `ConvertToYsonString` (or other name) CPO with proper extension into `yt/core` customisations.
6) Use CPO from (5) to eliminate direct dependency on `yt/core` of `TErrorAttribute` ctor.
7) Swap attributes implementation to the one which doesn’t use `IAttributeDictionary`.
8) At this point `stripped_error*` can be moved to library/cpp/yt and so can recursively dependant on THROW macro methods `Get/Find/…`.
9) Adjust CPO’s to work with `std::string` instead of `TYsonString` assuming text format to be used (maybe `TString` for now).
10) Remove dep of `library/cpp/yt/error` on `yson` entirely.
This pr addresses 5-6 steps of plan. Below is a brief explanation of design decisions. We also expressed everything related to key-value code in terms of aliases of `TErrorAttribute` so that later we could make a relatively simple switch to `std::string`. We expect to do steps 7-8 in the next pull request as everything should be ready
commit_hash:de9feca2bd24d823b33d904ef0fa5f4856f9b020
|
|
|
|
| |
commit_hash:60e005cdf76d5bff2a370a6b8f35ef4f6792f414
|
|
|
|
| |
commit_hash:81fb555c66d5ab8c5c260658be43bb7b61ab2622
|
|
|
|
| |
commit_hash:d73b9a2f472fbcf5f3ebc54787a964c15fba2504
|
|
|
|
| |
commit_hash:bbf24592c367fa158fbc53e041fe367374d0750e
|
|
|
|
| |
commit_hash:af73a3ea146e2eb6141cb19913e534d75ba573fd
|
|
|
|
| |
commit_hash:9c2ada5b95be3f810d2750406d74d839bd2598ff
|
|
|
|
| |
commit_hash:4591e7a8126c65672359f0cdee64ad789ca9ae74
|
|
|
|
| |
commit_hash:89b2dfdd1bb0d38b205f23744652b1db522e06c4
|
|
|
|
| |
commit_hash:1bcd4492a3f6a28a2be8e27b5660e98cc8e4c8d0
|
|
|
|
| |
commit_hash:1c25da3ce2125ca4d8d1e3e026ef0d2a6d0aa355
|
|
|
|
| |
commit_hash:b2cf3b80a36e4867b03a79f3e2c552f19920016f
|
|
|
|
| |
commit_hash:804530d1ee861ff42d7d8cad25d9f569b4feaacf
|
|
|
|
| |
commit_hash:a5695d6abae3e01238ce9328956579759caca454
|
|
|
|
| |
commit_hash:b6c853aa9dfce004915ca07c10eead963195eb97
|
|
|
|
| |
commit_hash:5ce8019253cdb971d1af36350e3efa3a4ec8545c
|
|
|
|
| |
commit_hash:a2bd81b4ecf17197f540e5096040491a78428687
|
|
|
|
| |
commit_hash:d8d2d7bd705c50833ff6e4b03fda17f1c97ca3b7
|
|
|
|
|
| |
[nodiff:caesar]
commit_hash:93eef00a7132e68e727aa1f4028616db6522d03c
|
|
|
|
| |
commit_hash:7313c6e8079a9821476cae610f095855b5c60c93
|
|
|
|
| |
commit_hash:e7480dc4037b4e009ef6dcc824697e6e89c5bfe1
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Y_ASSUME and Y_UNREACHABLE macros are platform-independent ways to
give a hint for a compiler about some value or state. Compilier may
use that hint for optimization. On the other hand if the hint is
wrong it may lead to undefined behavior.
YT_ASSUME and YT_UNREACHABLE are wrappers around Y_ASSUME and
Y_UNREACHABLE that actually check that the hint is correct in
debug build. That introduces some code safety and at the same time
allows certain optimization in optimized build.
commit_hash:2d0969361910a10a870bae226d838d494e656edb
|
|
|
|
| |
commit_hash:2edff041e77ead18a6bb7efeb13c8163d85c0750
|
|
|
|
| |
commit_hash:3381c362bf53465449cc36ad460347615208e001
|
|
|
|
|
| |
add constant for HTTP 434 code
commit_hash:bb04bc4efd36dc9989de7535b40c968c69b27472
|
|
|
|
| |
commit_hash:17dbd6d7e5fc440afa2d816e245a73e25135bfb5
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Plan:
1) Remove `IAttributedDictionary` type from the public API. \+
2) Remove `Set` method from public API in favor of `operator<<=`. \+
3) Adopt `ConvertTo<T>` (or other name) CPO with proper extension into `NYT::NYson::ConvertTo` from `yt/core`.
4) Use CPO from (3) to eliminate direct dependency on `yt/core` of `Get/Find` methods from attributes API.
5) Adopt `ConvertToYsonString` (or other name) CPO with proper extension into `yt/core` customisations.
6) Use CPO from (5) to eliminate direct dependency on `yt/core` of `TErrorAttribute` ctor.
7) Swap attributes implementation to the one which doesn’t use `IAttributeDictionary`.
8) At this point `stripped_error*` can be moved to library/cpp/yt and so can recursively dependant on THROW macro methods `Get/Find/…`.
9) Adjust CPO’s to work with `std::string` instead of `TYsonString` assuming text format to be used (maybe `TString` for now).
10) Remove dep of `library/cpp/yt/error` on `yson` entirely.
This pr addresses 3-4 steps of plan. Below is a brief explanation of design decisions.
We want to have a concept which detects if there is a `ConvertTo` method and if true, try calling it. Templates can only perform unqualified name lookup and if we allow non-ADL overloads to be found, we would have dependency on inclusion order (if `ConvertTo` is included prior to our code, everything works fine, but if the order is reverse, templated dispatch would fail, but direct call would work just fine). That is why we adopt niebloids which first disable ADL lookup of the name `ConvertTo` by directing it to niebloid implemented via `TagInvoke` mechanism. TagInvoke design <https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1895r0.pdf> . TL;DR: we want to have behavior which is consistent with inclusion order.
Key difference now is that `ConvertTo` works consistently in both manual calls and template function body calls and is no longer visible for ADL part of the unqualified name lookup
commit_hash:32af641bd0af559bfe670c2ceb36721fb4afc2dd
|
|
|
|
| |
commit_hash:ea73897466c683c4f852d5c9b6eebee419e6b8ab
|
|
|
|
|
| |
change base скрипт
commit_hash:abea0d7a031fd7bbbb1e754495e2c482b073be62
|
|
|
|
| |
commit_hash:ee52a32ff7f9601d2ea879be7d0bb07d0fe0f5b6
|
|
|
|
| |
commit_hash:f617f0a99f254f9100cea5bb7c2adaa30ae2038d
|
|
|
|
| |
commit_hash:7bbc13afbd569ca9bb064aed656c12644fd2de96
|
|
|
|
| |
commit_hash:46d59ab3acbd313753d3e46f3a6f10a8ebc424d8
|
|
|
|
| |
commit_hash:f76f3edfa160fba90d0f8d594211fa35c6eb6077
|
|
|
|
| |
commit_hash:fa9cb13adebcd335794779a600c4d88ad1af00d4
|
|
|
|
| |
commit_hash:95cd91e635cd58d38f4e69396d9d6d2201f1946b
|
|
|
|
|
| |
For example, this makes it possible to use gtest pointer matchers on smart pointers
commit_hash:2650074ae18ee35696b297d3d1f0393e7350789f
|
|
|
|
| |
commit_hash:7f7600d332c3ddb5c8372e921bcba3b4fbed68f8
|
|
|
|
| |
commit_hash:fdb84ff67180fd3887207a1b681ca8ab16652f47
|
|
|
|
| |
commit_hash:b51f9bd955778fca55a0628e2ba4acc60cd9c311
|