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

chore: RFC #2341 - 2020-04-15 - WASM Plugins Support #2341

Merged
merged 12 commits into from
May 28, 2020

Conversation

Hoverbear
Copy link
Contributor

@Hoverbear Hoverbear commented Apr 15, 2020

Signed-off-by: Ana Hobden <[email protected]>
@Hoverbear Hoverbear requested a review from a user April 15, 2020 17:21
@Hoverbear Hoverbear self-assigned this Apr 15, 2020
@Hoverbear Hoverbear changed the title RFC: WASM Foreign Module Support v0.1 feature: RFC - WASM Foreign Module Support v0.1 Apr 15, 2020
Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Ana Hobden <[email protected]>
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

@Hoverbear This is a great RFC!

rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
rfcs/2020-04-15-wasm-foreign-modules.md Outdated Show resolved Hide resolved
@Jeffail
Copy link
Contributor

Jeffail commented Apr 21, 2020

Looking good 👍

@Hoverbear Hoverbear linked an issue Apr 21, 2020 that may be closed by this pull request
@binarylogic binarylogic changed the title feature: RFC - WASM Foreign Module Support v0.1 chore: RFC - WASM Foreign Module Support v0.1 Apr 23, 2020
@binarylogic binarylogic changed the title chore: RFC - WASM Foreign Module Support v0.1 chore: RFC #2341 - 2020-04-15 - WASM Foreign Module Support Apr 23, 2020
Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Ana Hobden <[email protected]>
@Hoverbear Hoverbear changed the title chore: RFC #2341 - 2020-04-15 - WASM Foreign Module Support chore: RFC #2341 - 2020-04-15 - WASM Plugins Support May 7, 2020
@Hoverbear
Copy link
Contributor Author

@lukesteensen @binarylogic this is ready for merge. :)

Copy link
Member

@lukesteensen lukesteensen left a comment

Choose a reason for hiding this comment

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

Everything here sounds reasonable. Given the amount of engineering complexity and API surface area, I expect this feature to take a long time to settle. For the time being, I think we continue to focus on completing the thinnest possible vertical slice (parsing protobufs) and gain confidence in that before expanding the API at all. We have a lot of learning to do and getting out something super minimal to start playing with will help us do that.

@Hoverbear
Copy link
Contributor Author

This is still waiting for one approval :)

@Hoverbear
Copy link
Contributor Author

A little bird told me this is mergable so I'm merging it. 🎉

@Hoverbear Hoverbear merged commit 19dd0a0 into vectordotdev:master May 28, 2020
@binarylogic binarylogic added type: feature A value-adding code addition that introduce new functionality. and removed type: new feature labels Jun 16, 2020
mengesb pushed a commit to jacobbraaten/vector that referenced this pull request Dec 9, 2020
…tordotdev#2341)

* Add WASM RFC

Signed-off-by: Ana Hobden <[email protected]>

* Fix lints

Signed-off-by: Ana Hobden <[email protected]>

* Add title

Signed-off-by: Ana Hobden <[email protected]>

* Some rewording

Signed-off-by: Ana Hobden <[email protected]>

* Add lua benches

Signed-off-by: Ana Hobden <[email protected]>

* Touchup RFC

Signed-off-by: Ana Hobden <[email protected]>

* Add benchmarks of native and lua

Signed-off-by: Ana Hobden <[email protected]>

* Review and refine RFC

Signed-off-by: Ana Hobden <[email protected]>

* Move rfc

Signed-off-by: Ana Hobden <[email protected]>

* Update Registration docs

Signed-off-by: Ana Hobden <[email protected]>

* Remove files that weren't created.

Signed-off-by: Ana Hobden <[email protected]>
Signed-off-by: Brian Menges <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature A value-adding code addition that introduce new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WASM plugin architecture RFC
4 participants