Project

General

Profile

Actions

Bug #2742

closed

Face creation fails due to heavy application processing before calling processEvents

Added by Anonymous over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Category:
Base
Target version:
Start date:
04/10/2015
Due date:
% Done:

100%

Estimated time:

Description

Applications that have the following pattern:

Face face;

<load/process data>
<register prefixes>

face.processEvents()

fail because of the time gap between creating the Face and calling processEvents. This is a recurring problem for applications and repo-ng seems to be another victim.


Files

consumer.cpp (225 Bytes) consumer.cpp minimal program to reproduce Anonymous, 04/10/2015 11:55 AM
20150901233155.tgz (150 KB) 20150901233155.tgz Alex Afanasyev, 09/01/2015 05:12 PM

Related issues 1 (0 open1 closed)

Related to ndn-cxx - Task #2526: Application crashes after trying to put large data into faceClosed02/16/2015

Actions
Actions #1

Updated by Anonymous over 9 years ago

The following exception is thrown for the attached program:

libc++abi.dylib: terminating with uncaught exception of type ndn::Transport::Error: error while connecting to the forwarder

Could the Face constructor take a callback that informs the application of when it's ready to be used? Something similar to setInterestFilter's registration success and failure callbacks.

Actions #2

Updated by Junxiao Shi over 9 years ago

  • Category set to Base
Actions #3

Updated by Alex Afanasyev over 9 years ago

  • Status changed from New to In Progress
  • Assignee set to Alex Afanasyev
  • Target version set to v0.4
Actions #4

Updated by Junxiao Shi over 9 years ago

  • Related to Task #2526: Application crashes after trying to put large data into face added
Actions #5

Updated by Alex Afanasyev over 9 years ago

Integration tests are passing.

@Steve Will it be possible for you to verify if the original problem went away with the http://gerrit.named-data.net/#/c/2417/ commit?

Actions #6

Updated by Junxiao Shi over 9 years ago

  • Status changed from Code review to Resolved

Change is merged, but Steve needs to confirm the original issue is resolved.

Actions #7

Updated by Anonymous over 9 years ago

  • Status changed from Resolved to Closed

Everything looks good.

Sorry for the delay. The notification slipped by me.

Actions

Also available in: Atom PDF