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

Message ID filtering (SMSCGW to deal with messages that have certain message IDs) #273

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ovoo-unif
Copy link
Collaborator

Hi,

can you please review and consider this PR?

It is needed when trying to run 2 SMSCGW instances connected to cluster of 2 Cassandra nodes.

When SMSC checks for messages that should be re-sent, the instances should deal only with its 'own' messages (each SMSCGW has its own not-overlapping MsgID range).

Possibly it could be resolved in a better way. Anybody has any other ideas on how to achieve it?

Possibly this feature can be made configurable (to be turned on/off by configuration).

With kind regards,
adam

@gsaslis
Copy link
Contributor

gsaslis commented Jan 24, 2018

Hi @ovoo-unif !

Thanks for the PR ❤️ - appreciate you taking the time to contribute this!!

Could you please go through our Open Source Playbook and then proceed with signing our CLA, so we can then move forward with reviewing this?

Thanks in advance!
Yorgos

@ovoo-unif
Copy link
Collaborator Author

Hi Yorgos,

the CLA is signed.

Regards,
adam

@gsaslis
Copy link
Contributor

gsaslis commented Feb 6, 2018

Thanks @ovoo-unif !

pushing this for review from the @RestComm/messaging-squad

@gsaslis gsaslis requested a review from a team February 6, 2018 09:26
@vetss
Copy link
Contributor

vetss commented Mar 12, 2018

Hello @ovoo-unif

can you please explain me what is an advantage of usage of a single keyset for 2 SMSC GW instances in case you differentiate messages per a smsc gw instance (you want to smsc gw 2 will not use messages from ussd gw 1 and vice versa).

Differentiating of messageId is not an only problem. SMSC GW is designed now in a way when it thinks that it update a dtabase alone and nobody may update tables. If another server wants to change something in a database then it may confuse of a first server and lead collisions. If we want several SMSC GWs will reuse of a single keyspace we need to make a deep redesign of database usage and such usage means that messages may be loaded and delivered by a second server. IMO only this usage makes sense if we want to use same keyspace for a single keyspace.

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

Successfully merging this pull request may close these issues.

3 participants