Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

v3-dev #147

Merged
merged 45 commits into from May 21, 2016
Merged

v3-dev #147

merged 45 commits into from May 21, 2016

Conversation

mcollina
Copy link
Collaborator

@mcollina mcollina commented May 3, 2016

This pull requests deprecates node v0.10 and adds in node v6, moves from redis to ioredis #146, adds kafka support #144.

@behard @fullergalway please review and test.

@davedoesdev I am currently blocked on davedoesdev/qlobber-fsq#4.

@davedoesdev
Copy link
Collaborator

davedoesdev commented May 3, 2016

Yep, working on it (finishing up tests on Linux, testing on Windows). Should be a couple of days.

@davedoesdev
Copy link
Collaborator

qlobber-fsq v2.0.0 is now published:

  • changes fs-ext to an optional dependency
  • adds a single option when constructing to disable work queue semantics
  • auto-detects when fs-ext isn't available, emitting a single_disabled event

Also tested on Windows (without work queue semantics - tests modified to account for this case).

@coveralls
Copy link

coveralls commented May 5, 2016

Coverage Status

Coverage decreased (-0.8%) to 90.613% when pulling 57aa420 on v3-dev into 135dce3 on master.

@mcollina
Copy link
Collaborator Author

mcollina commented May 5, 2016

@davedoesdev can you please have a look at https://travis-ci.org/mcollina/ascoltatori/jobs/127988619#L1479? Can we get those removed here? Or it's fine?

@fullergalway
Copy link
Contributor

I will try to do some testing with a mosca and kafka ascoltatori today.

Regarding "Cannot find module 'fs-ext'" warnings for the fs ascoltatore, might this be fixed by adding fs-ext as a dependency in package.json...?

@mcollina
Copy link
Collaborator Author

mcollina commented May 5, 2016

It's in the optionalDependencies of qlobber-fsq, but it does not get compiled on node v6.

@davedoesdev
Copy link
Collaborator

@mcollina yes we can get rid of these. Add this line:

opts.single = false;

here: https://github.com/mcollina/ascoltatori/blob/master/lib/filesystem_ascoltatore.js#L20

@davedoesdev
Copy link
Collaborator

davedoesdev commented May 5, 2016

Btw, fs-ext should install on Node 6 (does on my box).
In any case, ascoltatori should set opts.single = false;

@fullergalway
Copy link
Contributor

Quick update: having some trouble testing kafka ascoltatori with mosca.

Mosca publishes metadata about user subscribes/unsubscribes etc. to $SYS* topics. Dollar sign is not allowed as kafka topic name so it breaks (uncaught error) when trying to create the topic. I'm unsure how best to resolve because a) there's no easy way to map non-allowed characters and allowed-characters, and b) I'm conflicted about the utility of many $SYS topics being created in my kafka(!).

I'm considering something like a Filter Ascoltatore which would be backed by two (or more?) delegate ascoltatores. This would allow me to filter all the $SYS topics to a different backend (eg., a file), while retaining the actual data topics in kafka.

Suggestions welcome.

@mcollina
Copy link
Collaborator Author

mcollina commented May 6, 2016

@fullergalway so the $SYS topics are needed 1) for stats and 2) by the persistence layer to synchronize the various instances of Mosca. Disabling stats is quite easy, while the second can cause issues.

I think a simple solution is to escape the $.

@coveralls
Copy link

coveralls commented May 6, 2016

Coverage Status

Coverage decreased (-0.8%) to 90.613% when pulling ee08b5b on v3-dev into 135dce3 on master.

@mcollina
Copy link
Collaborator Author

mcollina commented May 6, 2016

@psorowka would you like to take this for a spin before I release it as v3?

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.

None yet

5 participants