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

Add syncIndexes to all models JIC #1750

Merged
merged 1 commit into from
Sep 6, 2020

Conversation

Martii
Copy link
Member

@Martii Martii commented Sep 6, 2020

  • Possible recommendation as expires alias to expireAfterSeconds may need this to properly set values of an index that is created app side to sync with back-end.
  • Increase event emitter.setMaxListeners to accommodate additional MongoDB usage with error (warning) thrown:
Index event triggered/trapped for User model
Index event triggered/trapped for Discussion model
Index event triggered/trapped for Strategy model
Index event triggered/trapped for Remove model
Index event triggered/trapped for Comment model
Index event triggered/trapped for Group model
Index event triggered/trapped for Flag model
Index event triggered/trapped for Vote model
(node:10567) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 open listeners added to [NativeConnection]. Use emitter.setMaxListeners() to increase limit

Applies to #1744 #1730 and followup for #1516 ... followup needed on mLab for index that's already there (which I didn't put there) ... COMPLETED mLab followup... no errs.


NOTE:

* Possible recommendation as `expires` alias to `expireAfterSeconds` may need this to properly set values of an index that is created app side to sync with back-end.
* Increase event `emitter.setMaxListeners` to accommodate additional MongoDB usage with error thrown:

``` console
Index event triggered/trapped for User model
Index event triggered/trapped for Discussion model
Index event triggered/trapped for Strategy model
Index event triggered/trapped for Remove model
Index event triggered/trapped for Comment model
Index event triggered/trapped for Group model
Index event triggered/trapped for Flag model
Index event triggered/trapped for Vote model
(node:10567) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 open listeners added to [NativeConnection]. Use emitter.setMaxListeners() to increase limit
```

Applies to OpenUserJS#1744 OpenUserJS#1730 and followup for OpenUserJS#1516 ... followup needed on mLab for index that's already there *(which I didn't put there)*
@Martii Martii added DB Pertains inclusively to the Database operations. migration Use this to indicate that it may apply to an existing or announced migration. labels Sep 6, 2020
@Martii Martii merged commit 51becf5 into OpenUserJS:master Sep 6, 2020
@Martii Martii deleted the Issue-1744postFix branch September 6, 2020 04:39
@Martii
Copy link
Member Author

Martii commented Sep 6, 2020

This is peculiar...

stdout log:

Starting application...
Index event triggered/trapped for User model
Index event triggered/trapped for Discussion model
Index event triggered/trapped for Strategy model
Index event triggered/trapped for Remove model
Index event triggered/trapped for Comment model
Index event triggered/trapped for Group model
Index event triggered/trapped for Flag model
Index event triggered/trapped for Vote model
...
Index event triggered/trapped for Sync model
Index event triggered/trapped for User model
Index event triggered/trapped for Discussion model
...
Index event triggered/trapped for Strategy model
Index event triggered/trapped for Remove model
Index event triggered/trapped for Comment model
Index event triggered/trapped for Group model
Index event triggered/trapped for Flag model
Index event triggered/trapped for Vote model
...
Index event triggered/trapped for Script model

... some multiple, identical, events triggered from mongoose ? Perhaps will resolve itself in a dep update down the line. Will make a mental note of this to check OUJS as well.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DB Pertains inclusively to the Database operations. migration Use this to indicate that it may apply to an existing or announced migration.
Development

Successfully merging this pull request may close these issues.

1 participant