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

Looking for maintainers - please reach out! #1042

Open
raix opened this issue Dec 13, 2018 · 12 comments
Open

Looking for maintainers - please reach out! #1042

raix opened this issue Dec 13, 2018 · 12 comments

Comments

@raix
Copy link

raix commented Dec 13, 2018

The best packages are the ones maintained by those using it

@raix raix pinned this issue Dec 13, 2018
@jacky-cyber
Copy link

We are using the Meteor-CollectionFS, Thanks. I can maintain it.

@raix
Copy link
Author

raix commented Jan 2, 2019

Sounds great to have you on board @ljshj I'll add you to the repo - what is your Meteor login name (I'll add publish access)

@jacky-cyber
Copy link

My Meteor login name is ljshj too.

@raix
Copy link
Author

raix commented Jan 6, 2019

@ljshj I've added you to the organisation and meteor group - this should enable you to create new releases etc. let me know if it doesn't work - thanks!

@raix
Copy link
Author

raix commented Jan 9, 2019

@harryadel is also added as maintainer @ljshj - if you have questions along the way ping me or other contributors :)

@raix raix changed the title Looking for maintainers, I'm considering closing this project - please reach out! Looking for maintainers - please reach out! Jan 9, 2019
@harryadel
Copy link
Member

harryadel commented Jan 9, 2019

Hi @ljshj, Looking forward to doing great work with you! A review is needed here.

@jacky-cyber
Copy link

@harryadel Thanks. I will review it.

@jacky-cyber
Copy link

When the number of the file records is over 10,000 , the cpu usage will be very high, I want to modify its watch mode. @raix

@raix
Copy link
Author

raix commented Jan 10, 2019

@ljshj thats an issue with Meteor.Collection subscription right?

@jacky-cyber
Copy link

@raix I think so too. For example, https://github.com/CollectionFS/Meteor-CollectionFS/blob/bfd5852b257ae8b35c749f34ddb12c3b0cf73a19/packages/collection/common.js#L140, When the number of the file records is over 10,000 , the observe's performance is bad.

@raix
Copy link
Author

raix commented Jan 10, 2019

@ljshj ideally we would tap into the oplog query listener - I think @mitar amongst others ran into this issue.

@mitar
Copy link
Member

mitar commented Jan 10, 2019

@ljshj You mean the observe initialization performance is bad? BTW, do you need whole observe there or would it observeChanges be enough? For removing you probably just need _id, no? observeChanges provide you with that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants