Use only MemoryByteStore in analyzer_cli.

Use only MemoryByteStore in analyzer_cli.

Unfortunately evicting from cache makes it slow to use in potentially
short running cli tool.

R=brianwilkerson@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/e8c29784374b32b5d883fb45f51a8fe5bc5223fe

[Brian Wilkerson, 2017-02-16 20:41:13]

Could we use MemoryByteStore only when running in batch mode? Would that be
safe? (It should catch the "running tests" case.)

[scheglov, 2017-02-16 21:53:29]

On 2017/02/16 20:41:13, Brian Wilkerson wrote:
> Could we use MemoryByteStore only when running in batch mode? Would that be
> safe? (It should catch the "running tests" case.)

Running tests is a problem.
Using analyzer_cli to analyze small codebase and paying cost of running evicting
from cache is another problem.

For now I'd like to make it use of analysis driver, even if not persistent
caching.

[Brian Wilkerson, 2017-02-16 21:54:45]

Sounds reasonable. lgtm

[scheglov, 2017-02-16 22:07:48]

Description was changed from

==========
Use only MemoryByteStore in analyzer_cli.

Unfortunately evicting from cache makes it slow to use in potentially
short running cli tool.

R=brianwilkerson@google.com
BUG=
==========

to

==========
Use only MemoryByteStore in analyzer_cli.

Unfortunately evicting from cache makes it slow to use in potentially
short running cli tool.

R=brianwilkerson@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/e8c29784374b32b5d883fb45f51a8fe5bc5223fe
==========

[scheglov, 2017-02-16 22:07:49]

Committed patchset #1 (id:1) manually as
e8c29784374b32b5d883fb45f51a8fe5bc5223fe (presubmit successful).