Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(682)

Issue 2206783002: Stress test for comparing analysis performed in two AnalysisContext instances with different option… (Closed)

Created:
4 years, 4 months ago by scheglov
Modified:
4 years, 4 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Stress test for comparing analysis performed in two AnalysisContext instances with different options. It takes a Git repository, a folder in this repository and creates two AnalysisContext instances - with experimental and with stable analysis options. Then for each Git revision it applies the changes to Dart files in the folder to both contexts, analyzes and compares analysis cache results. The current implementation sets "finerGrainedInvalidation" to true for the experimental context. R=brianwilkerson@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/6f37f8521566e732a243279fbc98ab402ee82edc

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+639 lines, -1 line) Patch
M pkg/analyzer/lib/src/context/builder.dart View 2 chunks +4 lines, -1 line 0 comments Download
A pkg/analyzer/test/stress/for_git_repository.dart View 1 chunk +635 lines, -0 lines 1 comment Download

Messages

Total messages: 5 (2 generated)
scheglov
4 years, 4 months ago (2016-08-02 22:52:28 UTC) #1
Brian Wilkerson
The code duplication between this and the stress test in server makes me sad, but ...
4 years, 4 months ago (2016-08-03 13:49:11 UTC) #3
scheglov
4 years, 4 months ago (2016-08-03 16:07:08 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
6f37f8521566e732a243279fbc98ab402ee82edc (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698