Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Latest build of next f35bc939 produces slow binary #3

Closed
drewr opened this issue Oct 11, 2009 · 4 comments
Closed

Latest build of next f35bc939 produces slow binary #3

drewr opened this issue Oct 11, 2009 · 4 comments

Comments

@drewr
Copy link
Member

drewr commented Oct 11, 2009

Latest build takes 8s for a simple bal:

% ledger --version | head -1                        
Ledger next-0-g752677e, the command-line accounting tool
% time ledger bal -p this --no-total grocer         
            $ 147.36  Expenses:Food:Groceries
ledger bal -p this --no-total grocer  0.90s user 0.04s system 98% cpu 0.947 total
% ~/tmp/src/ledger/ledger --version | head -1
Ledger next-0-gf35bc93, the command-line accounting tool
% time ~/tmp/src/ledger/ledger bal -p this --no-total grocer
            $ 147.36  Expenses:Food:Groceries
~/tmp/src/ledger/ledger bal -p this --no-total grocer  7.83s user 0.08s system 98% cpu 8.001 total
@jwiegley
Copy link
Member

Can you run with "--verbose", which will show me how much time each piece is taking?

@drewr
Copy link
Member Author

drewr commented Oct 12, 2009

% ./ledger bal --verbose -p this --no-total grocer       
    0ms  [INFO]  Ledger starting
    2ms  [INFO]  Parsing file '/Users/aar/.ledgerrc'
    2ms  [INFO]  Parsing file '/Users/aar/.ledger'
 7874ms  [INFO]  Read journal file (7871ms)
 7874ms  [INFO]  Found 5721 transactions
            $ 147.36  Expenses:Food:Groceries
 8274ms  [INFO]  Finished executing command (399ms)
 8274ms  [INFO]  Ledger ended
% ./ledger --version | head -1
Ledger next-0-gf35bc93, the command-line accounting tool

@jwiegley
Copy link
Member

This sounds like a consequence of how I'm using Boost's time_facet logic. I'll have to optimize.

@jwiegley
Copy link
Member

Fixed.

This was referenced May 18, 2018
jwiegley pushed a commit that referenced this issue Jul 8, 2024
This issue was closed.
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

No branches or pull requests

2 participants