-
Notifications
You must be signed in to change notification settings - Fork 131
Adding an Avro backend #386
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
Open
ahasha
wants to merge
39
commits into
blaze:master
Choose a base branch
from
ahasha:avro
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
932bdde
Preliminary commit for Avro backend.
b2091f4
Adding test for avro to iterator
4cbe99f
Enabling read and write, adding append methods.
e07cfa7
Adding dependencies to build files
6cba475
Adding a convert edge from iterator back to avro
9ded2e1
Ensuring avro backend is registered on import
d7496ba
Fixing version problem with toolz
2adb2df
Adding support for array types
31ed159
Merge branch 'master' into avro
491c4d1
Attempting to fix travis build
2eeba0a
Reformatting test data
471becc
Adding conda selector for avro, since it is not Python 3 compatible
e05d1e1
Small fixes from code review.
31821d2
Converting tests to pytest idiom.
3773348
Changing import order to ensure test skip if avro not installed
2c0246a
Final small changes requested from code review
0d35449
Merge branch 'master' into avro
01b67a4
Use multipledispatch for schema discovery
a583e55
Bug fix for boolean types
eb004c7
Cleanup
58d4e6e
Adding dshape to avro schema generation, with doctest
64753e2
Bumping datashape version to fix build
d575ead
Merge branch 'master' into avro
5c2d082
Attempt to fix build for python3
78a591f
Whitespace error.
ee5e6f3
Python 3 support
ff32e9e
One more python3 bug I missed.
cbec204
Changing doctest not to be format sensitive.
4ebbf9b
Merge branch 'master' into avro
060c2b4
Updating avro type mappings
32900b9
Changing AVRO.uri attribute to AVRO.path, and allowing codec keyword …
95724cf
Dropping requirement for schema to be defined for new AVRO resource o…
76f2a72
Merge branch 'master' into avro
264fca2
Adding fastavro to meta.yaml
d9ca7ab
Merge branch 'master' into avro
1e56c1e
Correcting test failures due to incompatibility with pytest 3.0.3 as …
fd48794
Accidentally removed avro installation from travis.yml. Fixing
aba333a
Merge branch 'master' into avro
b66e7e3
Updating to new requirements tracking structure.
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| datashape >= 0.5.0 | ||
| numpy >= 1.7 | ||
| pandas >= 0.15.0 | ||
| toolz >= 0.7.3 | ||
| toolz >= 0.7.4 | ||
| multipledispatch >= 0.4.7 | ||
| networkx >= 1.0 | ||
| dask >= 0.11.1 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| fastavro==0.12.1 | ||
| avro==1.8.1; python_version < '3' | ||
| avro-python3==1.8.1; python_version > '2.7' |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do I need to do more than this to avoid breaking the build?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a separate
fastavropackage?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is... not sure why this didn't break the build. Should it be added here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This recipe affects building conda packages, which isn't actually used in testing. We would run into this issue upon releasing.