Skip to content

SLING-9389: Distribution Event Properties enhanced for holding queue item creation time stamp#7

Merged
cschneider merged 1 commit into
apache:masterfrom
harshchiki:issue/SLING-9389_1
Mar 28, 2022
Merged

SLING-9389: Distribution Event Properties enhanced for holding queue item creation time stamp#7
cschneider merged 1 commit into
apache:masterfrom
harshchiki:issue/SLING-9389_1

Conversation

@harshchiki
Copy link
Copy Markdown
Contributor

@harshchiki harshchiki commented May 27, 2020

In general,

  • Improvement aims at adding the queue item creation time, essentially when the the item was creation for the first time, and enqueue into the queue.
  • The purpose to get this detail is to be able to capture metrics at the consumer level. The consumers could have an event handler, which can capture the duration which turns out to be (NOW MINUS queue item creation time thrown in the distribution event package); NOW being the current time in the event handler (consumer).

@tmaret , @cschneider , kindly review.

@actinium15 , kindly review this PR - its identical to #2. I had to raise this new one to resolve some merge conflicts/issues.

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (1.8.0_241) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (1.8.0_252) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@cschneider cschneider merged commit 1215454 into apache:master Mar 28, 2022
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