[ 
https://issues.apache.org/jira/browse/CASSANDRA-16185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yasar Arafath Baigh updated CASSANDRA-16185:
--------------------------------------------
    Test and Documentation Plan: 
CommitLogMetrics Test-CasesCommitLogMetrics Test-Cases

 

1.  For below metrics, test-case are added in +*CommitLogTest.java*+ . In 
test-cases, mutations are added, which internally update metrics. 
 * *completedTasks*
 * *totalCommitLogSize*
 * *waitingOnCommit*

 

2. *pendingTasks* - In +*AbstractCommitLogService.java*+ , added 
testPendingTasks test-case,  since pendingTasks metric are incremented and 
decremented in single method of  *AbstractCommitLogService::maybeWaitForSync* .

So introduced a dummy method incrementPendingTaks in  
*FakeCommitLogService-class,* to manually update pendingTasks metric.

 

3. *waitingOnSegmentAllocation* - In +*CommitLogMetricsTest.java*+ test-case is 
added. In test *commitlog_segment_size_in_mb* is changed to *1mb*, and adding 
multiple  mutations so that waitingOnSegmentAllocation metric updated while 
creating new segments.

In normal case , waitingOnSegmentAllocation it will be zero, if during 
test-execution, waitingOnSegmentAllocation was not updated, then manually 
updating it.

  was:
CommitLogMetrics Test-CasesCommitLogMetrics Test-Cases

 

1.  For below metrics, test-case are added in +*CommitLogTest.java*+ . In 
test-cases, mutations are added, which internally update metrics. 
 * *completedTasks* 
 * *totalCommitLogSize* 
 * *waitingOnCommit*

 

2. *pendingTasks* - In +*AbstractCommitLogService.java*+ , added 
testPendingTasks test-case,  since pendingTasks metric are incremented and 
decremented in single method of  *AbstractCommitLogService::maybeWaitForSync* .

So introduced a dummy method incrementPendingTaks in  
*FakeCommitLogService-class,* to manually update pendingTasks metric.

 

3. *waitingOnSegmentAllocation* - In *CommitLogMetricsTest.java* test-case is 
added. In test *commitlog_segment_size_in_mb* is changed to *1mb*, and adding 
multiple  mutations so that waitingOnSegmentAllocation metric updated while 
creating new segments.

In normal case , waitingOnSegmentAllocation it will be zero, if during 
test-execution, waitingOnSegmentAllocation was not updated, then manually 
updating it.


> Add tests to cover CommitLog metrics
> ------------------------------------
>
>                 Key: CASSANDRA-16185
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16185
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Test/unit
>            Reporter: Benjamin Lerer
>            Assignee: Yasar Arafath Baigh
>            Priority: Normal
>             Fix For: 4.0-beta
>
>         Attachments: 0001-Unit-Test-cases-for-CommitLogMetrics.patch
>
>
> The only metrics that seems to be covered by unit test for the CommitLog 
> metrics is {{oversizedMutations}}. We should add testing the other ones.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to