Skip to content

Commit

Permalink
simplify logging
Browse files Browse the repository at this point in the history
  • Loading branch information
synzhu authored and huitseeker committed Sep 2, 2021
1 parent a916831 commit 4ce2800
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
6 changes: 1 addition & 5 deletions engine/common/synchronization/engine.go
Original file line number Diff line number Diff line change
Expand Up @@ -293,11 +293,7 @@ func (e *Engine) onSyncResponse(originID flow.Identifier, res *messages.SyncResp

// onBlockResponse processes a response containing a specifically requested block.
func (e *Engine) onBlockResponse(originID flow.Identifier, res *messages.BlockResponse) {
var blockIDs flow.IdentifierList
for _, block := range res.Blocks {
blockIDs = append(blockIDs, block.ID())
}
e.log.Debug().Str("origin_id", originID.String()).Strs("block_ids", blockIDs.Strings()).Msg("received block response")
e.log.Debug().Str("origin_id", originID.String()).Msg("received block response")
// process the blocks one by one
for _, block := range res.Blocks {
if !e.core.HandleBlock(block.Header) {
Expand Down
1 change: 1 addition & 0 deletions engine/common/synchronization/finalized_snapshot.go
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,7 @@ func (f *FinalizedHeaderCache) onFinalizedBlock(blockID flow.Identifier) {

// finalizationProcessingLoop is a separate goroutine that performs processing of finalization events
func (f *FinalizedHeaderCache) finalizationProcessingLoop() {
f.log.Debug().Msg("starting finalization processing loop")
notifier := f.finalizationEventNotifier.Channel()
for {
select {
Expand Down
2 changes: 1 addition & 1 deletion integration/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ integration-test: common-tests execution-tests verification-tests collection-tes

# NOTE: Currently skipping collection and consensus tests due to them not passing properly on CI
.PHONY: ci-integration-test
ci-integration-test: common-tests execution-tests epoch-tests verification-tests access-tests # collection-tests # consensus-tests
ci-integration-test: access-tests common-tests execution-tests epoch-tests verification-tests # collection-tests # consensus-tests

.PHONY: access-tests
access-tests:
Expand Down

0 comments on commit 4ce2800

Please sign in to comment.