What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

πŸ‘‹ should `EventConstnats::ACTIVATE_LOGGING` be enabled on production ?

UKGT7RC7P
UKGT7RC7P Posts: 538 πŸ§‘πŸ»β€πŸš€ - Cadet
edited June 2020 in Help

πŸ‘‹ should EventConstnats::ACTIVATE_LOGGING be enabled on production ?

Comments

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ LΓΆffelhardt Spryker Solution Partner Posts: 871 πŸ§‘πŸ»β€πŸš€ - Cadet

    @UKGT7RC7P please use Thread instead to write more messages πŸ™‚

    On production u have to enable this only if u have some problems and do u want investigate about it..

    But the part of JSON that u posted is not complete.. normally at the begin of json there is a listener class that failed...

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 πŸ§‘πŸ»β€πŸš€ - Cadet

    yes, i truncated the message

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 πŸ§‘πŸ»β€πŸš€ - Cadet

    so we can track down the errors in KIbana. Although only by searching for the listener name

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 πŸ§‘πŸ»β€πŸš€ - Cadet
    edited June 2020

    enabling the logging would add the context to the messages in RabbtMQ, but it also enables verbose logging I think

  • Ehsan Zanjani
    Ehsan Zanjani Head of Solution Architecture @ Spryker Posts: 113 πŸ§‘πŸ»β€πŸš€ - Cadet
    edited June 2020

    shouldΒ EventConstnats::ACTIVATE_LOGGINGΒ be enabled on production

    Enabling the logs will make the messages too fat (big stacktrace), this will impact the performance (mostly queue) if you have so many errors in production, but if the rate is low, you can enable it

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 πŸ§‘πŸ»β€πŸš€ - Cadet
    edited June 2020

    might be a nice thing to just include the message on it's own without stack trace, could then easily be correlated with the fat exception message in logs?

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 πŸ§‘πŸ»β€πŸš€ - Cadet

    i guess we can override the EventConsumer to do that

  • Ehsan Zanjani
    Ehsan Zanjani Head of Solution Architecture @ Spryker Posts: 113 πŸ§‘πŸ»β€πŸš€ - Cadet

    Yes I agree with you