Skip to content
This repository was archived by the owner on Nov 20, 2020. It is now read-only.

LZ4 compression support#219

Open
vchekan wants to merge 5 commits intojet:masterfrom
vchekan:master
Open

LZ4 compression support#219
vchekan wants to merge 5 commits intojet:masterfrom
vchekan:master

Conversation

@vchekan
Copy link
Copy Markdown

@vchekan vchekan commented Apr 12, 2018

Implements #129

Comment thread src/kafunk/Native/lz4.fs Outdated
@@ -0,0 +1,176 @@
namespace Kafunk.Native
#nowarn "9"
module Lz4Framing =
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should be internal?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done

Comment thread src/kafunk/Native/Loader.fs Outdated
namespace Kafunk.Native

/// Windlows pre-load dll from x86/x64 folder depending on Environment.Is64BitProcess
module Loader =
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can also make internal, rather than namespace do module internal Kafunk.NativelLoader

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done

Comment thread RELEASE_NOTES.md Outdated
@@ -1,3 +1,6 @@
### 0.1.17 - 12.4.2018
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's do an alpha first 0.1.17-alpha1

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants