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.

Sign up for GitHub

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

Jump to bottom

Mining Analysis #429

Merged
merged 68 commits into from
Sep 30, 2022
Merged

Mining Analysis #429

merged 68 commits into from
Sep 30, 2022

Conversation

temaniarpit27
Copy link
Contributor

@temaniarpit27 temaniarpit27 commented Jun 13, 2022 •
edited
Loading

This PR will introduce the metrics in the mining module and bor consensus module via opentelemetry

@linear
Copy link

linear bot commented Jun 13, 2022

POS-505 Opentelemetry for Bor

vcastellm
vcastellm reviewed Jun 15, 2022
View reviewed changes
miner/worker.go Outdated Show resolved Hide resolved
@temaniarpit27 temaniarpit27 marked this pull request as ready for review June 15, 2022 11:59
JekaMas
JekaMas reviewed Jun 16, 2022
View reviewed changes
consensus/bor/bor.go Outdated Show resolved Hide resolved
JekaMas
JekaMas reviewed Jun 16, 2022
View reviewed changes
consensus/bor/bor.go Outdated Show resolved Hide resolved
@temaniarpit27
Copy link
Contributor Author

@JekaMas i have reverted your commit
anon-91 seems fine after deploying the PR

@manav2401 manav2401 marked this pull request as draft September 1, 2022 05:41
@temaniarpit27 temaniarpit27 marked this pull request as ready for review September 8, 2022 07:23
manav2401 and others added 15 commits September 23, 2022 20:31
@manav2401
merge develop
97e2840
@temaniarpit27
change heimdall version to develop
aa532ee
@manav2401
miner/worker: fix duplicate call to tx ordering
9bdbc1e
@manav2401
Merge branch 'arpit/pos-505' of https://github.com/maticnetwork/bor i…
b8bc0b6 
…nto arpit/pos-505
@manav2401
Merge branch 'develop' of https://github.com/maticnetwork/bor into ar…
b726e53 
…pit/pos-505
@manav2401
miner/worker: refactor tracing
9b4f796
@manav2401
consensus/bor: use tracing package
b626090
@manav2401
tracing: add more abstraction for spans
4324423
@manav2401
tracing: set all attributes at once
a810072
@temaniarpit27
Merge branch 'develop' into arpit/pos-505
0ab5821
@manav2401
remove nested tracing from blockchain.WriteBlockAndSetHead function
54fef9c
@manav2401
remove nested tracing from statedb.IntermediateRoot function
c99de20
@manav2401
handle end span in bor.Seal function
b6ff133
@manav2401
fix: typo
91388bf
@manav2401
minor fixes
f4a8a86
@manav2401 manav2401 merged commit 2a6ea47 into develop Sep 30, 2022
@manav2401 manav2401 deleted the arpit/pos-505 branch September 30, 2022 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manav2401 manav2401 manav2401 approved these changes

@JekaMas JekaMas JekaMas approved these changes

@vcastellm vcastellm vcastellm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@temaniarpit27 @codecov-commenter @manav2401 @JekaMas @ssandeep @vcastellm @0xsharma