damccorm commented on PR #40096:
URL: https://github.com/apache/beam/pull/40096#issuecomment-5636647616

   Oh that's true, I was looking at the wrong log line as the base. With that 
said, I think it is actually because of the fetch depth. The initial checkout 
does fetch-depth=1 by default 
(https://github.com/actions/checkout/blob/f548e57e544e1ff5a4c46bf1e1b8685f8e4a348a/action.yml#L74)
   
   On a fresh machine, I tried:
   
   - `git clone https://github.com/apache/beam --depth=1`
   - `time git fetch origin pr-bot-state` (took 1m10s)
   
   Then I tried:
   
   - `git clone https://github.com/apache/beam --depth=1`
   - `time git fetch origin pr-bot-state --depth=10` (took 5s) - `--depth=1` is 
even faster
   
   So I think if we want to drive down fetch times, we should just reduce the 
fetch depth.
   
   Since we don't have real locking semantics, I'm interested in keeping the 
bot state as close as possible to the real state.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to