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.

Sign up for GitHub

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

Jump to bottom

[WIP] Release 0.5.0 #155

Merged
merged 6 commits into from
Nov 27, 2016
Merged

[WIP] Release 0.5.0 #155

merged 6 commits into from
Nov 27, 2016

Conversation

Ianleeclark
Copy link
Owner

@Ianleeclark Ianleeclark commented Nov 25, 2016 •
edited
Loading

@Ianleeclark
Issue #150
3273079 
CHANGE:
  - Moved all redis stuff into `./server/peerStore`
  - peerStore/redisStore.go Now creates connectiosn on-the-fly. This is a
    degradation for performance and shouldn't be necessary. However, I'm lazy
    and Redis is going to be trashed for Olivia. In the future, a better
    solution would be to send all redis operations to a central store and have
    that store process via one connection.
@Ianleeclark
Issue #150
bc80fec 
CHANGE:
  - server/peerStore/redis_test.go Importing `server` package, as we need the
    definitions from it.
@Ianleeclark
Issue #150
a879c23 
Major refactor of server/annoucne -> announce/announce.go
@Ianleeclark
Issue #150
b47e4b5 
CHANGE:
  - server/peerStore/peerstore.go Added a new method all peerStores should
    implement.
  - server/peerStore/redisStore.go Added new wrapper methods to implement the
    `PeerStore` interface.
  - server/server.go Still attempting to decouple heavily spaghettied code.
@Ianleeclark
Issue #150
790826d 
CHANGE:
  - kvStoreInterfaces/redis.go Now houses common redis calls used
    **throughout** the application. The `server/peerStore/redisStore.go` file
    just implements a wrapper around these.
@Ianleeclark
Merge pull request #153 from GrappigPanda/issue-150
dfe17f4 
[WIP] Fixes #150
@Ianleeclark Ianleeclark merged commit dfe17f4 into master Nov 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Ianleeclark