TrueCloudLab/frostfs-api-go

Author	SHA1	Message	Date
Evgenii Stratonikov	5e1c6a908f	[#111 ] protogen: Emit slice of messages without a pointer All checks were successful DCO action / DCO (pull_request) Successful in 41s Details Tests and linters / Tests (1.22) (pull_request) Successful in 55s Details Tests and linters / Tests (1.23) (pull_request) Successful in 57s Details Tests and linters / Tests with -race (pull_request) Successful in 1m8s Details Tests and linters / Lint (pull_request) Successful in 2m12s Details ``` goos: linux goarch: amd64 pkg: git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/refs cpu: 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz │ old │ new │ │ sec/op │ sec/op vs base │ ObjectIDSlice/0_elements/to_grpc_message-8 3.193n ± 2% 3.242n ± 0% +1.50% (p=0.034 n=10) ObjectIDSlice/0_elements/from_grpc_message-8 3.197n ± 2% 3.343n ± 1% +4.57% (p=0.000 n=10) ObjectIDSlice/0_elements/marshal-8 5.666n ± 3% 5.642n ± 0% -0.42% (p=0.000 n=10) ObjectIDSlice/1_elements/to_grpc_message-8 53.10n ± 6% 29.78n ± 12% -43.92% (p=0.000 n=10) ObjectIDSlice/1_elements/from_grpc_message-8 28.99n ± 5% 29.77n ± 7% ~ (p=0.165 n=10) ObjectIDSlice/1_elements/marshal-8 49.08n ± 7% 50.72n ± 6% ~ (p=0.218 n=10) ObjectIDSlice/50_elements/to_grpc_message-8 1652.5n ± 7% 277.2n ± 1% -83.22% (p=0.000 n=10) ObjectIDSlice/50_elements/from_grpc_message-8 261.2n ± 11% 226.7n ± 15% -13.19% (p=0.003 n=10) ObjectIDSlice/50_elements/marshal-8 1.512µ ± 6% 1.514µ ± 6% ~ (p=0.955 n=10) geomean 52.15n 39.99n -23.31% │ old │ new │ │ B/op │ B/op vs base │ ObjectIDSlice/0_elements/to_grpc_message-8 0.000 ± 0% 0.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/0_elements/from_grpc_message-8 0.000 ± 0% 0.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/0_elements/marshal-8 0.000 ± 0% 0.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/1_elements/to_grpc_message-8 32.00 ± 0% 24.00 ± 0% -25.00% (p=0.000 n=10) ObjectIDSlice/1_elements/from_grpc_message-8 24.00 ± 0% 24.00 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/1_elements/marshal-8 48.00 ± 0% 48.00 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/50_elements/to_grpc_message-8 1.578Ki ± 0% 1.250Ki ± 0% -20.79% (p=0.000 n=10) ObjectIDSlice/50_elements/from_grpc_message-8 1.250Ki ± 0% 1.250Ki ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/50_elements/marshal-8 2.000Ki ± 0% 2.000Ki ± 0% ~ (p=1.000 n=10) ¹ geomean ² -5.62% ² ¹ all samples are equal ² summaries must be >0 to compute geomean │ old │ new │ │ allocs/op │ allocs/op vs base │ ObjectIDSlice/0_elements/to_grpc_message-8 0.000 ± 0% 0.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/0_elements/from_grpc_message-8 0.000 ± 0% 0.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/0_elements/marshal-8 0.000 ± 0% 0.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/1_elements/to_grpc_message-8 2.000 ± 0% 1.000 ± 0% -50.00% (p=0.000 n=10) ObjectIDSlice/1_elements/from_grpc_message-8 1.000 ± 0% 1.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/1_elements/marshal-8 1.000 ± 0% 1.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/50_elements/to_grpc_message-8 51.000 ± 0% 1.000 ± 0% -98.04% (p=0.000 n=10) ObjectIDSlice/50_elements/from_grpc_message-8 1.000 ± 0% 1.000 ± 0% ~ (p=1.000 n=10) ¹ ObjectIDSlice/50_elements/marshal-8 1.000 ± 0% 1.000 ± 0% ~ (p=1.000 n=10) ¹ geomean ² -40.18% ² ¹ all samples are equal ² summaries must be >0 to compute geomean ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-28 11:53:08 +03:00
Evgenii Stratonikov	a2025376fc	[#111 ] proto/test: Add repeated message test Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-28 11:53:08 +03:00
Evgenii Stratonikov	5fece80b42	[#108 ] protogen: Distinguish between empty and nil messages All checks were successful DCO action / DCO (pull_request) Successful in 39s Details Tests and linters / Tests (1.22) (pull_request) Successful in 59s Details Tests and linters / Tests (1.23) (pull_request) Successful in 57s Details Tests and linters / Tests with -race (pull_request) Successful in 1m9s Details Tests and linters / Lint (pull_request) Successful in 2m12s Details Refs #59 Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-27 11:02:18 +03:00
Evgenii Stratonikov	981dc785f3	[#107 ] proto/test: Add oneof test Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-26 14:36:19 +03:00
Evgenii Stratonikov	866db105ed	[#107 ] protogen: Unify oneof getters with default protoc plugin Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-26 14:36:19 +03:00
Evgenii Stratonikov	937a53683a	[#107 ] protogen: Fix oneof JSON marshaling Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-26 14:36:19 +03:00
Evgenii Stratonikov	4a00ef946f	[#107 ] protogen: Fix oneof getter scalar default value Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-26 14:36:19 +03:00
Evgenii Stratonikov	61f6f0f4a2	[#107 ] proto/test: Regenerate *.pb.go Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-26 14:36:19 +03:00
Evgenii Stratonikov	c1b5f46f98	[#107 ] protogen: Remove unused parameter Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-26 14:36:19 +03:00
Evgenii Stratonikov	7be31eb847	[#77 ] protogen: Add tests for JSON format Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-19 10:47:00 +03:00
Evgenii Stratonikov	adb7c602d7	[#77 ] protogen: Initial implementation Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-19 10:47:00 +03:00
Evgenii Stratonikov	a28ceb251a	[#77 ] util/proto: Optimize int32 marshaling This is the approach used in easyproto `52d3ac4744/writer.go (L203)` It allows to occupy slightly less space for negative numbers. The format is still protobuf, although, technically, this is a breaking change for our stable marshaling format. However, we don't use int32 at all and all enums have positive values, so nothing is broken. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-16 17:11:13 +03:00
Evgenii Stratonikov	47a48969b0	[#103 ] proto: Test end-to-end scenario All checks were successful DCO action / DCO (pull_request) Successful in 1m27s Details Tests and linters / Tests (1.19) (pull_request) Successful in 1m36s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m43s Details Tests and linters / Lint (pull_request) Successful in 1m51s Details Tests and linters / Tests with -race (pull_request) Successful in 2m6s Details Test the generated code, do not write yet another marshaling routine in tests. Before: ``` ok git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/util/proto 0.003s coverage: 55.6% of statements ``` After: ``` ok git.frostfs.info/TrueCloudLab/frostfs-api-go/v2/util/proto 0.003s coverage: 80.0% of statements ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-09 11:18:17 +03:00
Evgenii Stratonikov	19247e8941	[#103 ] protogen: Handle uint32 type Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-09 11:18:17 +03:00
Evgenii Stratonikov	ff4f31b6f3	[#103 ] protogen: Handle files in all packages Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-09 11:18:17 +03:00
Anton Nikiforov	b72aa14bab	[#87 ] proto: Process files with `protoc` version `27.2` Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-08-06 17:20:29 +03:00
Airat Arifullin	9b90d139c5	[#94 ] object: Generate protobufs for `Patch` method * Generate protobufs for patch method; * Create marshalers, unmarshalers, converters for gererated types; * Add unit-tests. Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-07-29 08:55:14 +00:00
Evgenii Stratonikov	f517e39491	[#91 ] protogen: Support unpacked repeated uint64 fields All checks were successful DCO action / DCO (pull_request) Successful in 1m18s Details Tests and linters / Lint (pull_request) Successful in 1m31s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m39s Details Tests and linters / Tests (1.19) (pull_request) Successful in 1m45s Details Tests and linters / Tests with -race (pull_request) Successful in 2m21s Details Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-07-16 14:39:20 +03:00
Evgenii Stratonikov	610c450a65	[#90 ] proto/test: Fix go vet warnings Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-07-15 16:40:21 +03:00
Evgenii Stratonikov	3f92d7bfb0	[#90 ] proto/test: Fix proto file formatting Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-07-15 16:40:21 +03:00
Evgenii Stratonikov	1772b92182	[#69 ] object: Initial EC implementation All checks were successful DCO action / DCO (pull_request) Successful in 1m7s Details Tests and linters / Tests (1.19) (pull_request) Successful in 1m37s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m33s Details Tests and linters / Lint (pull_request) Successful in 1m46s Details Tests and linters / Tests with -race (pull_request) Successful in 1m45s Details Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-03-19 15:23:01 +03:00
Evgenii Stratonikov	a85146250b	[#68 ] Makefile: Use gofumpt for formatting All checks were successful DCO action / DCO (pull_request) Successful in 1m17s Details Tests and linters / Tests (1.20) (pull_request) Successful in 2m22s Details Tests and linters / Lint (pull_request) Successful in 2m51s Details Tests and linters / Tests (1.19) (pull_request) Successful in 2m48s Details Tests and linters / Tests with -race (pull_request) Successful in 2m40s Details Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-03-14 10:31:58 +03:00
Dmitrii Stepanov	634e24aba7	[#65 ] object: Fix SetMarshalData for PutSingle request All checks were successful DCO action / DCO (pull_request) Successful in 1m11s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m25s Details Tests and linters / Tests (1.19) (pull_request) Successful in 1m33s Details Tests and linters / Tests with -race (pull_request) Successful in 1m56s Details Tests and linters / Lint (pull_request) Successful in 1m14s Details Allow to reset marshal data Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-02-15 15:44:01 +03:00
Alexander Chuprov	72885aae83	[#62 ] signature: Refactor BufferPool All checks were successful Tests and linters / Tests (1.19) (pull_request) Successful in 7m46s Details DCO action / DCO (pull_request) Successful in 8m16s Details Tests and linters / Tests (1.20) (pull_request) Successful in 9m24s Details Tests and linters / Tests with -race (pull_request) Successful in 9m25s Details Tests and linters / Lint (pull_request) Successful in 9m49s Details Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-01-12 18:09:28 +03:00
Evgenii Stratonikov	b46e8cfbda	[#59 ] util: Rename stableMarshaler All checks were successful DCO action / DCO (pull_request) Successful in 1m23s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m46s Details Tests and linters / Lint (pull_request) Successful in 1m59s Details Tests and linters / Tests (1.19) (pull_request) Successful in 2m7s Details Tests and linters / Tests with -race (pull_request) Successful in 2m22s Details It should be with a single `l`, see `json.Marshaler`. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-11-21 15:28:03 +03:00
Evgenii Stratonikov	9cc2095446	[#59 ] util: Restore backwards compatibility in NestedStructureMarshal() Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-11-21 15:27:57 +03:00
Dmitrii Stepanov	f50872f1bc	[#58 ] object: Allow to set marshal data Now it is possible set marshaled data to reduce memory allocations. Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2023-10-31 13:47:48 +03:00
Evgenii Stratonikov	17bed735a1	[#20 ] Update api version All checks were successful DCO action / DCO (pull_request) Successful in 1m14s Details Tests and linters / Tests (1.19) (pull_request) Successful in 1m39s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m42s Details Tests and linters / Lint (pull_request) Successful in 1m57s Details Tests and linters / Tests with -race (pull_request) Successful in 2m5s Details Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-08-17 11:10:12 +03:00
Airat Arifullin	9a7b47d769	[#50 ] util: Use protowire for util/proto helpers * Use methods to calculate size Signed-off-by: Airat Arifullin a.arifullin@yadro.com	2023-08-10 16:40:53 +00:00
Evgenii Stratonikov	3072090c77	[#51 ] protogen: Do not panic in StableSize() All checks were successful Tests and linters / Lint (pull_request) Successful in 1m56s Details Tests and linters / Tests (1.20) (pull_request) Successful in 1m7s Details Tests and linters / Tests (1.19) (pull_request) Successful in 2m37s Details Tests and linters / Tests with -race (pull_request) Successful in 1m44s Details Refs TrueCloudLab/frostfs-node#554 . Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-08-02 10:47:14 +03:00
Evgenii Stratonikov	7a5ee927c8	[#49 ] util/proto: Do not allocate in StringSize() All checks were successful Tests and linters / Tests (1.19) (pull_request) Successful in 1m1s Details Tests and linters / Lint (pull_request) Successful in 2m10s Details Tests and linters / Tests with -race (pull_request) Successful in 2m18s Details Tests and linters / Tests (1.20) (pull_request) Successful in 2m52s Details It was not catched by the test because most of the time the function is inlined. However, I've seen it allocating with pprof in one of the earlier builds. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-07-26 18:52:59 +03:00
Evgenii Stratonikov	6e92d7d5de	[#49 ] util/proto: Make `NestedStructure*` generic Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-07-26 18:31:23 +03:00
Evgenii Stratonikov	849de02bc3	[#49 ] util/proto: Calculate repeated field size without allocations Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-07-26 18:31:23 +03:00
Evgenii Stratonikov	59c8421597	[#49 ] util/proto: Use StableSize() to determine if the struct is empty `reflect` is not necessary here, and checking `StableSize` is what we _want_ anyway. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-07-26 18:31:00 +03:00
Dmitrii Stepanov	68021b910a	[#38 ] signature: Increase pool max object size According to the results of profiling, objects with a size of 72KB are mainly allocated. Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2023-06-02 17:27:16 +03:00
Evgenii Stratonikov	e9e7901f67	[#27 ] util/proto: Fix staticcheck warning Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-05-03 14:15:56 +03:00
Evgenii Stratonikov	2aa3ee46e7	[#27 ] util/signature: Fix staticcheck warning Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2023-05-03 14:15:56 +03:00
Pavel Karpy	d9347a05f0	[#21 ] *: Drop reputation system Signed-off-by: Pavel Karpy <p.karpy@yadro.com>	2023-04-17 15:46:52 +03:00
Dmitrii Stepanov	ec0d0274fa	[#3 ] signature: Add buffer pool Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2023-03-10 13:36:26 +03:00
Dmitrii Stepanov	73fde0e37c	[#3 ] signature: Verify parts in parallel Verify request/response parts in parallel Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2023-03-10 13:36:26 +03:00
Dmitrii Stepanov	d005bf0393	[#3 ] signature: Refactor sign and verify Split methods to separate files, drop redundant intefaces Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2023-03-10 13:36:26 +03:00
Alex Vanin	f69d2ad83c	Rename package name Due to source code relocation from GitHub. Signed-off-by: Alex Vanin <a.vanin@yadro.com>	2023-03-07 13:42:36 +03:00
Evgenii Stratonikov	1351b6656d	Move to frostfs-api Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2022-12-12 17:40:48 +03:00
Leonard Lyubich	cf868188ef	[#418 ] netmap: Support `NetmapService.NetmapSnapshot` RPC Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>	2022-09-19 16:44:34 +04:00
Evgenii Stratonikov	ea7a3811bf	[#415 ] *: Add maintenance status and node state Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>	2022-09-16 18:50:53 +04:00
Evgenii Stratonikov	9e17cdfc76	[#413 ] util/proto: Marshal repeated []byte fields correctly Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>	2022-08-27 11:06:58 +03:00
Evgenii Stratonikov	dd233c3fbc	[#386 ] util/signature: Add WalletConnect API support To avoid introducing new dependency (neo-go), crypto routines are used as in other code. Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>	2022-06-21 20:09:33 +03:00
Evgenii Stratonikov	454b5c0ed7	[#406 ] protogen: Uncomment control service line Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>	2022-06-20 14:45:58 +03:00
fyrchik	0b8e4e4753	[#400 ] Support latest API in protoc plugin (#400 ) Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>	2022-05-30 10:33:01 +03:00
Evgenii Stratonikov	77a3ba9197	[#393 ] util/signature: Remove bytes pool and provide buffer explicitly Chained verification is done in a single thread there is no need to use pool to do this. Also because newly allocated items are 5 MiB in size we can run out of memory given that typical header it much less in size. Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>	2022-04-21 15:11:57 +03:00

1 2

81 commits