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

EVA: enable concurrent transfers to the same peer #7698

Closed
drew2a opened this issue Nov 20, 2023 · 1 comment
Closed

EVA: enable concurrent transfers to the same peer #7698

drew2a opened this issue Nov 20, 2023 · 1 comment

Comments

@drew2a
Copy link
Contributor

drew2a commented Nov 20, 2023

EVA protocol is a binary data transfer protocol based on TFTP with window size (RFC 7440) implemented on top of the ipv8 networking layer.

The design initially aimed to facilitate a thumbnail feature for channels, with the first version of the protocol kept as simple as possible. However, this protocol has evolved over time.

The first round of improvements was spearheaded by @devos50, following his experiment with large-scale binary model transfers for distributed machine learning experiments (#6831). This enhancement also incorporated a bug fix identified by @rmadhwal and detailed in trustchain-superapp issue #101.

The next set of refinements was led by @kozlovsky in #6845, following his review of #6831.

A possible future enhancement is enabling concurrent transfers to the same peer, as discussed in #7074 (comment), #7258 (comment). Until now, this feature wasn't a priority in Tribler, given the lack of substantial demand for simultaneous binary transfers to the same peer. However, the time may be right to reconsider this?

It's also important to note that the Python EVA and Kotlin EVA have different implementations and are likely incompatible as of now.

@qstokkink
Copy link
Contributor

The code for this feature has been removed and this issue is no longer relevant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants