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

Implement a new torrent notifier #178

Closed
Ianleeclark opened this issue Dec 11, 2016 · 0 comments
Closed

Implement a new torrent notifier #178

Ianleeclark opened this issue Dec 11, 2016 · 0 comments

Comments

@Ianleeclark
Copy link
Owner

This will be for #170

Essentially we need either a non-blocking channel or pg_notify.

The non-blocking channel should be the implementation for MySQL whereas pg_notify is, naturally, for postgres.

Whenever WhiteTorrent.AddWhitelistedTorrent() is called (or any method of adding a new torrent into the database), the notification handler ought to be notified. The PG implementation ought to be much simpler, as channels won't be passed around.

@Ianleeclark Ianleeclark mentioned this issue Dec 11, 2016
7 tasks
Ianleeclark added a commit that referenced this issue Dec 17, 2016
ADD:
  - announce/impl/postgresCatcher.go Added to offer an implementation of a
    notifier. MySQL still needs to be added.
Ianleeclark added a commit that referenced this issue Dec 17, 2016
Added tests to the new postgresCatcher and the postgres_libpq
Ianleeclark added a commit that referenced this issue Dec 17, 2016
Updating build/postgres_create.sql to create the white_torrents table and add a
trigger and updating the .travis.yml file to use the new postgres_create file
Ianleeclark added a commit that referenced this issue Dec 17, 2016
Updating. travis.yml. File had wrong name for postgres create
Ianleeclark added a commit that referenced this issue Dec 17, 2016
CHANGE:
  - .travis.yml Had incorrect `-` before the `postgres_create.sql` script line
Ianleeclark added a commit that referenced this issue Dec 18, 2016
CHANGE:
  - database/mysql/database_test.go Now doesnt drop WhiteTorrents table.
Ianleeclark added a commit that referenced this issue Dec 18, 2016
CHANGE
  - .travis.yml now uses an input file rather than `<` the .sql file.
Ianleeclark added a commit that referenced this issue Dec 18, 2016
ADD:
  - build/postgres_create.sql Updating script to create the `white_torrents`
    table.
Ianleeclark added a commit that referenced this issue Dec 18, 2016
ADD:
  - .travis.yml Added postgres version 9.4 because travis defaults to 9.1
Ianleeclark added a commit that referenced this issue Dec 18, 2016
ADD:
  - announce/impl/mysqlCatcher.go Handles serving new white torrents added to
    the mysql implementation.
Ianleeclark added a commit that referenced this issue Dec 18, 2016
CHANGE:
  - announce/impl/*.go:NewCatcher changed to `New{MySQL,Postgres}Catcher` to
    better indicate which implementation will be built.
Ianleeclark added a commit that referenced this issue Dec 18, 2016
* Issue #178

ADD:
  - announce/impl/postgresCatcher.go Added to offer an implementation of a
    notifier. MySQL still needs to be added.

* Issue #178

Added tests to the new postgresCatcher and the postgres_libpq

* Issue #178

Updating build/postgres_create.sql to create the white_torrents table and add a
trigger and updating the .travis.yml file to use the new postgres_create file

* Issue #178

Updating. travis.yml. File had wrong name for postgres create

* Issue #178

CHANGE:
  - .travis.yml Had incorrect `-` before the `postgres_create.sql` script line

* Issue #178

CHANGE:
  - database/mysql/database_test.go Now doesnt drop WhiteTorrents table.

* Issue #178

CHANGE
  - .travis.yml now uses an input file rather than `<` the .sql file.

* Issue #178

ADD:
  - build/postgres_create.sql Updating script to create the `white_torrents`
    table.

* Issue #178

ADD:
  - .travis.yml Added postgres version 9.4 because travis defaults to 9.1

* Issue #178

ADD:
  - announce/impl/mysqlCatcher.go Handles serving new white torrents added to
    the mysql implementation.

* Issue #178

CHANGE:
  - announce/impl/*.go:NewCatcher changed to `New{MySQL,Postgres}Catcher` to
    better indicate which implementation will be built.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant