search

OfficeDev / microsoft-teams-apps-company-communicator

Company Communicator app template
MIT License
330 stars 403 forks source link

Sending Messages takes 3x the time than prior to updating to v5.5.2 #1564

Closed bbruenings closed 1 month ago

bbruenings commented 2 months ago

Hello together,

after we did the Update to the v5.5.2 of the Communicator, sending a message takes about 3x of what it used to be before the update.

Prior to v5.5.2 image

After the update to v5.5.2 image

There you can see that the SendMessage Function now takes around 4,3 seconds per message. It used to be around 1.5 before the update.

This a big issue for our customer as there company-wide messages now take up to 12 hours to be send.

I hope you can help us resolve the issue.

I'm happy to share what ever you need.

Thanks and best regards Bennet

MSFT-klpinhac commented 2 months ago

Please note: a large number of people are in focus for this instance of the company communicator, so the time it takes to send the messages does actually have significant impact

peddivyshnavi commented 2 months ago

Hi @bbruenings Thanks for reaching out to us! we did'nt added any logic to send functionality while upgrading to new version we have added new functionality like Adaptive cards

bbruenings commented 2 months ago

Hi @peddivyshnavi Thanks for confirming. That also surprised me that there was no change there but we still see the change after the Deployment. Is there anything I can provide to further investigate the issue? As of my knowledge there is nothing else that changed during the update.

peddivyshnavi commented 2 months ago

Hi @bbruenings As mentioned earlier we didnt added any logic to send functionality .we will analyze further and will update you

bbruenings commented 2 months ago

Hi @peddivyshnavi

I double checked which version we were coming from on production.

The update not only included the newest version 5.5.2 but also 5.5.1 and 5.5. as we were coming from 5.4.1.

Sorry for the confusion 5.5 included changes to the send feature as it included the scheduled message feature correct?

Thanks and best regards Bennet

peddivyshnavi commented 2 months ago

Hi @bbruenings we were checking on this we will update you as early as possible

peddivyshnavi commented 2 months ago

HI @bbruenings Schedule send is separate component in send functionality we didnt added or merged schedule send with send functionality

peddivyshnavi commented 1 month ago

@bbruenings Could you please update us on the status of the issue?

bbruenings commented 1 month ago

@peddivyshnavi The issue still persist. We will use the next Week to recreate the setup on our DEV Instance and try to get some more insights on what is going on. I will keep you updated with new information as soon as I have them.

peddivyshnavi commented 1 month ago

Sure we will check accordingly

peddivyshnavi commented 1 month ago

we were closing ADO ticket for avoiding duplicate and we will continue discussing this issue over here. thanks

microsoft-github-policy-service[bot] commented 1 month ago

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.