[Findit] Pull changelogs in batch.

[Findit] Pull changelogs in batch.

Use 'name-status' flag to get the json changelogs in a regression range, instead
of download every changelogs in a regression range and parse the changelogs.

For example:
https://chromium.googlesource.com/chromium/src/+log/52.0.2725.0..52.0.2726.0?...

Committed: https://chromium.googlesource.com/infra/infra/+/9182172995eeae65cc98d48ceea4d42e372fc1bd

[Sharu Jiang, 2016-05-06 06:33:54]

katesonia@chromium.org changed reviewers:
+ mbarbella@google.com, stgao@chromium.org

[Sharu Jiang, 2016-05-06 06:33:54]

[Findit] Pull changelogs in batch.

Use 'name-status' flag to get the json changelogs in a regression range, instead
of download every changelogs in a regression range and parse the changelogs.

For example:
https://chromium.googlesource.com/chromium/src/+log/52.0.2725.0..52.0.2726.0?...

[Sharu Jiang, 2016-05-06 06:34:22]

Description was changed from

==========
PUll changelogs in batch.


intermediate.

BUG=
==========

to

==========
[Findit] Pull changelogs in batch.

Use 'name-status' flag to get the json changelogs in a regression range, instead
of download every changelogs in a regression range and parse the changelogs.

For example:
https://chromium.googlesource.com/chromium/src/+log/52.0.2725.0..52.0.2726.0?...
==========

[Sharu Jiang, 2016-05-06 06:34:32]

PTAL :)

[Sharu Jiang, 2016-05-06 06:38:10]

Patchset #1 (id:1) has been deleted

[stgao, 2016-05-06 06:59:18]

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
File appengine/findit/common/git_repository.py (right):

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
appengine/findit/common/git_repository.py:261: 'name-status': '1'})
Will this include the file change?

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
File appengine/findit/common/test/git_repository_test.py (right):

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
appengine/findit/common/test/git_repository_test.py:590: url = args[1]
Can we expand the arguments above instead?

[Sharu Jiang, 2016-05-06 17:45:58]

Patchset #2 (id:40001) has been deleted

[Sharu Jiang, 2016-05-06 17:46:13]

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
File appengine/findit/common/git_repository.py (right):

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
appengine/findit/common/git_repository.py:261: 'name-status': '1'})
On 2016/05/06 06:59:18, stgao wrote:
> Will this include the file change?

Yes, it includes the 'tree-diff's

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
File appengine/findit/common/test/git_repository_test.py (right):

https://codereview.chromium.org/1957653002/diff/20001/appengine/findit/common...
appengine/findit/common/test/git_repository_test.py:590: url = args[1]
On 2016/05/06 06:59:18, stgao wrote:
> Can we expand the arguments above instead?

Done.

[stgao, 2016-05-06 17:54:23]

lgtm

[Martin Barbella, 2016-05-06 18:16:32]

mbarbella@chromium.org changed reviewers:
+ mbarbella@chromium.org

[Martin Barbella, 2016-05-06 18:16:32]

lgtm

[stgao, 2016-05-06 19:33:12]

The CQ bit was checked by stgao@chromium.org

[commit-bot: I haz the power, 2016-05-06 19:33:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1957653002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1957653002/60001

[commit-bot: I haz the power, 2016-05-06 19:40:11]

Description was changed from

==========
[Findit] Pull changelogs in batch.

Use 'name-status' flag to get the json changelogs in a regression range, instead
of download every changelogs in a regression range and parse the changelogs.

For example:
https://chromium.googlesource.com/chromium/src/+log/52.0.2725.0..52.0.2726.0?...
==========

to

==========
[Findit] Pull changelogs in batch.

Use 'name-status' flag to get the json changelogs in a regression range, instead
of download every changelogs in a regression range and parse the changelogs.

For example:
https://chromium.googlesource.com/chromium/src/+log/52.0.2725.0..52.0.2726.0?...

Committed:
https://chromium.googlesource.com/infra/infra/+/9182172995eeae65cc98d48ceea4d...
==========

[commit-bot: I haz the power, 2016-05-06 19:40:12]

Committed patchset #2 (id:60001) as
https://chromium.googlesource.com/infra/infra/+/9182172995eeae65cc98d48ceea4d...