Untitled

Activity

SAKTRUNK-66 1

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author 0m    
Reviewer - 100% reviewed 50m 1 Spent this afternoon comparing kernel with the patch vs w...
Reviewer - 0% reviewed      
Reviewer - 0% reviewed      
Reviewer - 0% reviewed      
Reviewer - 100% reviewed 35m    
Reviewer - 0% reviewed      
Reviewer - 0% reviewed      
Reviewer - Complete 3m    
Reviewer - 0% reviewed      
Total   1h 29m 1  
#permalink

Objectives

There are no specific objectives for this review.

Branches in review

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

Brian Baillargeon

Spent this afternoon comparing kernel with the patch vs without the patch. Th...

Spent this afternoon comparing kernel with the patch vs without the patch. The patch looks logically sound to me; it seems to omit the unnecessary inserts while leaving all other behavior unchanged.

/kernel-impl/.../impl/DbAuthzGroupService.java Changed  
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

X
Assign To Me

Log time against