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

Explain LINKER_ERROR #5702

Merged
merged 5 commits into from
Dec 4, 2022
Merged

Explain LINKER_ERROR #5702

merged 5 commits into from
Dec 4, 2022

Conversation

0xbe1
Copy link
Contributor

@0xbe1 0xbe1 commented Nov 26, 2022

fix #5663

Description

Test Plan

Copy link
Contributor

@clay-aptos clay-aptos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you again!

@banool
Copy link
Contributor

banool commented Nov 30, 2022

This is a good start but I'd like more detail. This doesn't really tell users what is specifically wrong and how to debug it. Perhaps someone from the Move team could help? cc @movekevin

@banool
Copy link
Contributor

banool commented Nov 30, 2022

One example of a LINKER_ERROR: If you try to call a function that does not exist on chain. For example, if you call a module that you have the source for locally, but have not published that module on chain, it’ll fail when you try to actually run it.

@clay-aptos
Copy link
Contributor

I'm noting we should also port these updates back to the source files.

@gregnazario
Copy link
Contributor

LINKER_ERROR actually has a bunch of possible reasons, but I think this helps either way for now cc @wrwg if he knows anyone who can give a more complete answer.

Thanks for adding this!

banool
banool previously requested changes Dec 3, 2022
Copy link
Contributor

@banool banool left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry but "The function that your command is executing is not a valid function. Double-check your command" is not very useful, it is far too vague. What does valid mean? What do we mean by command? I don't see the value in landing this as is.

@0xbe1
Copy link
Contributor Author

0xbe1 commented Dec 3, 2022

@banool agreed! I have improved the case that I know of combining your input. Can you take another look? Thanks

Copy link
Contributor

@davidiw davidiw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, adjusted

@davidiw davidiw dismissed banool’s stale review December 3, 2022 23:40

manually addressed

@davidiw davidiw enabled auto-merge (squash) December 3, 2022 23:40
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@davidiw davidiw disabled auto-merge December 4, 2022 00:08
@davidiw davidiw merged commit d0cef70 into aptos-labs:main Dec 4, 2022
@0xbe1 0xbe1 deleted the patch-8 branch December 4, 2022 00:14
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2022

✅ Forge suite land_blocking success on e004af979afeb8023c084b3c727afc435fdf4b63

performance benchmark with full nodes : 6919 TPS, 5739 ms latency, 9300 ms p99 latency,no expired txns
Test Ok

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2022

✅ Forge suite compat success on testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> e004af979afeb8023c084b3c727afc435fdf4b63

Compatibility test results for testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> e004af979afeb8023c084b3c727afc435fdf4b63 (PR)
1. Check liveness of validators at old version: testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b
compatibility::simple-validator-upgrade::liveness-check : 7091 TPS, 5464 ms latency, 7500 ms p99 latency,no expired txns
2. Upgrading first Validator to new version: e004af979afeb8023c084b3c727afc435fdf4b63
compatibility::simple-validator-upgrade::single-validator-upgrade : 5128 TPS, 7830 ms latency, 9600 ms p99 latency,no expired txns
3. Upgrading rest of first batch to new version: e004af979afeb8023c084b3c727afc435fdf4b63
compatibility::simple-validator-upgrade::half-validator-upgrade : 4474 TPS, 8882 ms latency, 12400 ms p99 latency,no expired txns
4. upgrading second batch to new version: e004af979afeb8023c084b3c727afc435fdf4b63
compatibility::simple-validator-upgrade::rest-validator-upgrade : 6766 TPS, 5791 ms latency, 9400 ms p99 latency,no expired txns
5. check swarm health
Compatibility test for testnet_2d8b1b57553d869190f61df1aaf7f31a8fc19a7b ==> e004af979afeb8023c084b3c727afc435fdf4b63 passed
Test Ok

@Markuze Markuze mentioned this pull request Dec 5, 2022
areshand pushed a commit to areshand/aptos-core-1 that referenced this pull request Dec 18, 2022
* Explain LINKER_ERROR

fix aptos-labs#5663

* Add hypen

* update LINKER_ERROR description

* update LINKER_ERROR description

* Update developer-docs-site/docs/guides/handle-aptos-errors.md

Co-authored-by: Clay Murphy <[email protected]>
Co-authored-by: David Wolinsky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Mention LINKER_ERROR in "Handle Errors in Aptos"
5 participants