Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stack]: redesign errors - 0 #2147

Open
wants to merge 39 commits into
base: main
Choose a base branch
from
Open

[stack]: redesign errors - 0 #2147

wants to merge 39 commits into from

Conversation

tusharmath
Copy link
Contributor

@tusharmath tusharmath commented Jun 9, 2024

/fixes #1508

  • Drop color from core::Errata
  • GRPC Error Snapshot shouldn't have changed.
  • Drop thiserror::Error from Errata. It is not an error so remove it from places where it's being used like one.
  • Drop thiserror::Error from everywhere.
image

@github-actions github-actions bot added the type: chore Routine tasks like conversions, reorganization, and maintenance work. label Jun 9, 2024
Copy link

codecov bot commented Jun 9, 2024

Codecov Report

Attention: Patch coverage is 74.72973% with 187 lines in your changes missing coverage. Please review.

Project coverage is 85.84%. Comparing base (f464794) to head (6caaabe).

Files Patch % Lines
src/core/errata.rs 87.72% 34 Missing ⚠️
tailcall-cloudflare/src/cache.rs 0.00% 20 Missing ⚠️
tailcall-cloudflare/src/file.rs 0.00% 19 Missing ⚠️
src/core/ir/error.rs 63.26% 18 Missing ⚠️
tailcall-cloudflare/src/http.rs 0.00% 11 Missing ⚠️
tailcall-typedefs/src/main.rs 0.00% 7 Missing ⚠️
src/cli/generator/config.rs 0.00% 5 Missing ⚠️
src/cli/generator/generator.rs 0.00% 5 Missing ⚠️
tailcall-aws-lambda/src/runtime.rs 0.00% 4 Missing ⚠️
src/cli/tc.rs 0.00% 3 Missing ⚠️
... and 33 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2147      +/-   ##
==========================================
- Coverage   85.92%   85.84%   -0.08%     
==========================================
  Files         227      235       +8     
  Lines       21792    21777      -15     
==========================================
- Hits        18724    18694      -30     
- Misses       3068     3083      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mehulmathur16 and others added 2 commits June 9, 2024 17:58
Co-authored-by: mehulmathur001 <160306774+mehulmathur001@users.noreply.github.com>
Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
@tusharmath tusharmath marked this pull request as ready for review June 11, 2024 06:41
# Conflicts:
#	Cargo.toml
# Conflicts:
#	src/core/http/request_context.rs
#	src/core/ir/io.rs
Copy link

Action required: PR inactive for 2 days.
Status update or closure in 5 days.

@github-actions github-actions bot added the state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. label Jun 14, 2024
@mehulmathur16 mehulmathur16 removed the state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. label Jun 14, 2024
@tusharmath tusharmath changed the title chore: Redesign Errors [stack]: redesign errors - 0 Jun 14, 2024
tusharmath and others added 2 commits June 15, 2024 18:23
# Conflicts:
#	src/core/app_context.rs
#	src/core/ir/cache.rs
#	src/core/ir/eval.rs
#	src/core/ir/io.rs
#	src/core/ir/mod.rs
@github-actions github-actions bot added the ci: benchmark Runs benchmarks label Jun 15, 2024
# Conflicts:
#	src/core/app_context.rs
#	src/core/config/transformer/type_merger/type_merger.rs
#	src/core/config/transformer/type_name_generator.rs
#	src/core/generator/from_json.rs
#	src/core/generator/from_proto.rs
#	src/core/generator/json/field_base_url_generator.rs
#	src/core/generator/json/schema_generator.rs
#	src/core/http/request_context.rs
#	src/core/ir/cache.rs
#	src/core/ir/eval.rs
#	src/core/ir/eval_http.rs
#	src/core/ir/mod.rs
#	tests/expression_spec.rs
Copy link

github-actions bot commented Jun 17, 2024

🐰Bencher

ReportWed, June 26, 2024 at 20:00:16 UTC
Projecttailcall
Branch2147/merge
Testbedbenchmarking-runner
Click to view all benchmark results
BenchmarkLatencyLatency Results
nanoseconds (ns) | (Δ%)
Latency Upper Boundary
nanoseconds (ns) | (%)
from_json_bench✅ (view plot)7,198,300.00 (+0.94%)7,260,258.57 (99.15%)
group_by✅ (view plot)544.35 (-3.87%)631.63 (86.18%)
input/args.missing✅ (view plot)25.92 (+7.46%)27.40 (94.61%)
input/args.nested.existing✅ (view plot)53.54 (+12.23%)61.95 (86.41%)
input/args.nested.missing✅ (view plot)36.36 (-4.48%)40.90 (88.90%)
input/args.root✅ (view plot)48.84 (+10.78%)59.01 (82.77%)
input/headers.existing✅ (view plot)32.85 (+3.83%)33.86 (97.02%)
input/headers.missing✅ (view plot)31.55 (+2.70%)33.68 (93.67%)
input/value.missing✅ (view plot)23.41 (-0.12%)24.99 (93.68%)
input/value.nested.existing✅ (view plot)40.12 (-3.51%)45.09 (88.98%)
input/value.nested.missing✅ (view plot)36.98 (+0.85%)38.67 (95.63%)
input/value.root✅ (view plot)35.59 (-6.44%)41.59 (85.59%)
input/vars.existing✅ (view plot)7.45 (-4.32%)9.09 (81.95%)
input/vars.missing✅ (view plot)10.96 (+9.67%)13.42 (81.68%)
synth_nested✅ (view plot)20,740.00 (-0.88%)21,504.97 (96.44%)
test_batched_body✅ (view plot)2,675.70 (-99.00%)1,553,931.34 (0.17%)
test_batched_body #2✅ (view plot)1,624,600.00 (-4.45%)1,822,807.85 (89.13%)
test_data_loader✅ (view plot)470,690.00 (+0.36%)485,386.71 (96.97%)
test_handle_request✅ (view plot)152,620.00 (-2.28%)171,619.09 (88.93%)
test_http_execute_method✅ (view plot)17,898.00 (-1.44%)19,260.68 (92.93%)
with_mustache_expressions✅ (view plot)1,185.10 (+1.61%)1,231.45 (96.24%)
with_mustache_literal✅ (view plot)769.47 (+6.64%)777.71 (98.94%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

Copy link

Action required: PR inactive for 2 days.
Status update or closure in 5 days.

@github-actions github-actions bot added state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. and removed state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. labels Jun 19, 2024
Copy link

Action required: PR inactive for 2 days.
Status update or closure in 5 days.

@github-actions github-actions bot added state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. and removed state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. labels Jun 21, 2024
@tusharmath tusharmath removed the ci: benchmark Runs benchmarks label Jun 22, 2024
Co-authored-by: mehulmathur001 <160306774+mehulmathur001@users.noreply.github.com>
Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
@github-actions github-actions bot added the ci: benchmark Runs benchmarks label Jun 26, 2024
@mehulmathur16 mehulmathur16 removed the ci: benchmark Runs benchmarks label Jun 26, 2024
@github-actions github-actions bot added the ci: benchmark Runs benchmarks label Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci: benchmark Runs benchmarks type: chore Routine tasks like conversions, reorganization, and maintenance work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Format runtime errors using CLIError
3 participants