| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Release-Notes: skip
Change-Id: Ic99c0d13a99f9577f0762e3c8bd41ca2b798f744
|
|
|
|
|
| |
Release-Notes: skip
Change-Id: Id6feef84ad0578f2a09c0b97591d8be768293507
|
|\ |
|
| |
| |
| |
| |
| | |
Change-Id: Id9ec22f0688adf290b27f32ffb71d63df3f6a763
Release-Notes: skip
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When receiving the email, knowing the change owner lets the receiver
knows it is one of their change and might refine reviewing priority for
some people.
We had that customization for a few years now, and I felt like it can
be upstreamed (ref: https://phabricator.wikimedia.org/T43608 ).
Release-Notes: Emails notifications for comments now mentions who owns the change.
Change-Id: If676e4f1b75bdbe2d39dcfe1fbe841520e81068f
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
It has not been possible for change owners to index their own changes
for a while, but the documentation still claims that they can.
Bug: Issue 337070245
Release-Notes: Skip
Change-Id: I67a973da01b57c5cd779314132c7e586c92f6c9b
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Note that the underscore (`_`) has been removed from the variable name
to avoid resemblance to the prefix `_` used in `_number`, which
(according to the documentation) denotes deprecation.
Follow-Up-To: I85c946ef
Bug: Issue 325309573
Release-Notes: Add `virtual_id_number` to `ChangeInfo`
Forward-Compatible: checked
Change-Id: Iddf5dee45f80f863ec8dbc113e15e9499bfee757
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
As long as we have Rest API and tests for label functions we will use
this method and the compiler warnings makes it difficult to check
if new changes introduces other compiler warnings.
Release-Notes: skip
Change-Id: I788ef20091b39d72712ecfaed287a15af5921001
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Any code inside the orElse() will always be called, even if the result
isn't used because the optional is present. This is wasteful at a
minimum and can be actively harmful (as seen in [1]) because of side
effects or performance impacts. Fix that by replacing all orElse() calls
that create new instances or do non-constant work with either a call to
orElseGet() or an if statement.
It's possible this will result in performance improvements, but I didn't
attempt to measure any, so I'm not including a release note.
It would be nice if there were an ErrorProne checker for this, but one
doesn't exist yet.
[1] https://gerrit-review.googlesource.com/c/gerrit/+/417915/comment/0ea287cd_bfecb7f2/
Change-Id: Icab4115998fb4f3787ca74c0890949cb3ce5c6a6
Release-Notes: skip
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We need to set properties, not attributes for paper-tooltip.
Google-Bug-Id: b/333416564
Release-Notes: skip
Change-Id: I48d41184d10aa0b2fd719717c38664792fa8bc54
(cherry picked from commit 4f7449246c08c9d6066ebbf5dca54423d07a9afc)
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* stable-3.6:
Reset pageResultSize when PaginationType NONE back-fill results
Add test paginationType NONE to query only if there are more results
Fix paginationType NONE to run further queries only if needed
Run tests in FakeQueryChangesTest for paginationType NONE
Fix detection of invalid SSH key algorithm
Demonstrate SshKeyUtil fails to validate invalid SSH keys
Move comment of PaginatingSource.read()
Fix visibility filter for changes when paginationType NONE
Add tests for pagination to back fill the results
Allow plugins to use PredicateArgs
Release-Notes: skip
Change-Id: I78775c62d60bbdb63eb19dd8683179091c6ca447
|
| |\ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* stable-3.5:
Reset pageResultSize when PaginationType NONE back-fill results
Add test paginationType NONE to query only if there are more results
Fix paginationType NONE to run further queries only if needed
Run tests in FakeQueryChangesTest for paginationType NONE
Fix detection of invalid SSH key algorithm
Demonstrate SshKeyUtil fails to validate invalid SSH keys
Move comment of PaginatingSource.read()
Fix visibility filter for changes when paginationType NONE
Add tests for pagination to back fill the results
Allow plugins to use PredicateArgs
Release-Notes: skip
Change-Id: I8e166780d069267e3f25c454dc947ebe1d32d949
|
| | |\ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* stable-3.4:
Reset pageResultSize when PaginationType NONE back-fill results
Release-Notes: skip
Change-Id: I32a22fcbd7c84d07ff96df44975dd4c53259b94e
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is a follow up of change 413358.
In particular, if some results are skipped because of
of the visibility constraints, more changes need to
be asked from the index, and pageResultSize needs to
be reset to know how many results are returned from each
query.
Without this change querying for more changes could end
up in an infinite loop as exposed in a test in 3.6 [1]
[1]: https://gerrit.googlesource.com/gerrit/+/refs/heads/stable-3.6/javatests/com/google/gerrit/server/query/change/FakeQueryChangesTest.java#126
Release-Notes: skip
Change-Id: I566010c6f5bfcb4fbc003bc6693aa25d4dd44a81
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
If the first query to the index did not fill the full limit the caller
wants, we paginate to obtain more results. However, subsequent queries
are not needed if there are no more results from the previous query.
This add a test for the change 413358 [1], that has reintroduced the
issue fixed in change 344695.
[1]: https://gerrit-review.googlesource.com/c/gerrit/+/413358/comment/85bca522_2b3552b2/
Bug: Issue 330195358
Release-Notes: skip
Change-Id: If578ee877587488fdf3715331bd8d23e9d86f750
|
| | |\| |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* stable-3.4:
Fix paginationType NONE to run further queries only if needed
Fix detection of invalid SSH key algorithm
Demonstrate SshKeyUtil fails to validate invalid SSH keys
Move comment of PaginatingSource.read()
Fix visibility filter for changes when paginationType NONE
Add tests for pagination to back fill the results
Release-Notes: skip
Change-Id: I9df29c9ae3b37026c9866fd97b6bb7f8dbb0dde9
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
If the first query to the index did not fill the full limit the caller
wants, we paginate to obtain more results. However, subsequent queries
are not needed if the previous query returned less results
than the limit.
This is a follow up of the change 413358 [1], that has reintroduced the
issue fixed in change 344695.
[1]: https://gerrit-review.googlesource.com/c/gerrit/+/413358/comment/85bca522_2b3552b2/
Bug: Issue 330195358
Release-Notes: Fix pagination to stop querying when there are no more results
Change-Id: Ie326f3628f5312a83bf83177a3fa5134f042b59a
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Verify that the OpenSSH key algorithm matches the one
associated with the public key. Throw a new specific
InvalidKeyAlgorithmException if the two algorithms do not match.
This is a breaking change because invalid OpenSSH keys
have been tolerated since the very beginning, when Shawn introduced
the SSH support in Change 6610.
Attempting to store new invalid SSH keys would fail and result
an HTTP status 400 error as response to the add keys REST-API.
Make SshKeyCacheImpl tolerant to existing keys stored
in the accounts' profile, otherwise Gerrit may start flagging
keys that have been previously stored as invalid, resulting
in random authentication failures by existing users.
Existing invalid keys are reported in the error_log with the
associated exceptions and automatically fixed, removing the
invalid key from the accounts profile and adjusting the key
algorithm with the one associated with the public key.
Bug: Issue 330758152
Release-Notes: Breaking change: validate and reject SSH keys with invalid or mismatched algorithm
Change-Id: I83c89a786f70aa3b88744a70f10012415f45f284
(cherry picked from commit 6eac4fe62a6a081c5c9395f8874bdc49615eea0d)
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The SshKeyUtil has always missed the validation of the SSH key algo
specified as a prefix of the Base-64 encoded key.
Whilst the behaviour has always been the same since 2008, it is
nonetheless buggy and should be validated for preventing the storage
of invalid keys.
TODO: Mark the SSH key validation test as disabled for allowing the
build to succeed. The test can be enabled back again once the validation
has been amended to verify the key algorithm.
Bug: Issue 330758152
Release-Notes: skip
Change-Id: I42b1c6474fa876828e5353e81e97b21b981665f9
(cherry picked from commit 4f3ff5abdb18ad34078d8dd2f0ad1d4e610957d1)
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Move the comment about restarting the DataSource for back filling
results skipped for visibility. The comment in fact is valid for
any PaginationType specified.
Adress the comment in change 413358 [1].
[1]: https://gerrit-review.googlesource.com/c/gerrit/+/413358/comment/d7a90bb0_0f8ebc9f/
Release-Notes: skip
Change-Id: I9826b8be4005a946c7952ae89eb81f8c191997e3
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The introduction of paginationType NONE in
change 379354 affected the API pagination.
In particular, if some results are skipped because of
of the visibility constraints, more changes need to
be asked from the index, until we have filled the
full limit the caller wants or if there are no more
changes [1].
[1]: https://gerrit-review.googlesource.com/c/gerrit/+/379354/8/java/com/google/gerrit/index/query/PaginatingSource.java#72
Bug: Issue 328958027
Release-Notes: Fix NONE paginationType filtering of changes
Change-Id: I3ccff7f8ba5a6d3903f9acbe3f5c439c55225ce2
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When querying Lucene, if we have not filled the full
limit the caller wants, `PaginatingSource.read()`
need to restart the source and continue. This should happen
regardless of the paginationType.
Bug: Issue 328958027
Release-Notes: skip
Change-Id: I02bc97eeecde2149e374dd036b87dd4c06034f27
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The tests in FakeQueryChangesTest were executed only
for the default pagination (OFFSET) and SEARCH_AFTER
Release-Notes: skip
Change-Id: Ic5c84bdc28fccbf0a4c931d7137ce8f232907622
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Change visibility of the PredicateArgs constructor to public. This will
allow plugins implementing operators with args to use PredicateArgs
instead of implementing their own arg parser.
Release-Notes: skip
Change-Id: Ief39f4606a96c24ae3ad58bd0031a825ce4cdcab
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Adds a new feature enabled by the new core.trustLooseRefStat config key
which can be set to `after_open` in order to invalidate NFS attribute
caching before trusting the dir/file attributes of loose refs and their
parent directories. This has the benefit of fixing some multi-primary
race conditions at the downside of some performance impacts.
Additionally using the SnapshottingRefDirectory via Gerrit's
usePerRequestRefCache will limit the NFS invalidation done during a
single request and mitigates all the major performance impacts of the
`after_open` setting (some minor performance impacts are still expected).
In order to update to this JGit version we also need to bump the Bazel
version to 6.3.2 (originally done in change I993fb2b36) in order to get
the ijar fix necessary to update to the newer libraries (originally done
in changes I62c90995 and I0ad4adce) needed by this JGit version.
This also includes some other minor JGit fixes.
$ git log --oneline --no-merges acf21c0bc...c0b415fb0
81d64c913 Cache refreshed loose ref dirs in SnapshottingRefDirectory
72fa0a53e TreeRevFilter: correct changedPathFilter usage for multi-paths inclusion
9e841dd4a PathFilterGroup: implement getPathsBestEffort()
e5a17df8f CleanupService: preload JgitText if not running in OSGi
c1eba8abe DfsPackFile: get commitGraph.readChangePaths from repo config
631cbf59f CommitGraphLoader: receive readChangedPaths as parameter
900c5e62b Add pull request template discouraging usage of pull requests
dc4fa3906 Update CONTRIBUTING.md to point to GitHub issues
328008aa2 FS_POSIX.runInShell(): on MacOS use a login shell
da7a88bce [ssh] Implement the "Ciphers" SSH config
bf7dd9add ShutdownHook: run on bundle deactivation if in OSGi
819c5bcc8 [diffmergetool] Fix running command on Cygwin
24307bd38 [releng] Bump japicmp base version to 6.9.0.202403050737-r
fe681b34f DfsBlockCache: move cache table specific implementations to a new class
07d7452b3 Prepare 6.10.0-SNAPSHOT builds
e61c20b1e Prepare 6.9.1-SNAPSHOT builds
d764bc34f JGit v6.9.0.202403050737-r
058faa28f JGit v6.9.0.202403050045-r
d69a6c5de Prepare 6.9.0-SNAPSHOT builds
21f7fdff7 Introduce core.trustLooseRefStat config
407ccef1c JGit v6.9.0.202402281855-rc1
64911e211 Update SECURITY.md
5c94dcc56 DfsObjDatabase: Let object database instantiate DfsPackFiles
049749558 DfsPackFile: Abstract the bitmap loading to support other backends
d132050c2 Remove unused API problem filters
a44b9e8bf Support public key in IdentityFile
f11d1365f Revert "StartGenerator: Fix parent rewrite with non-default RevFilter"
55f3a9586 Prepare 6.9.0-SNAPSHOT builds
36dd4dd9b JGit v6.9.0.202402211805-m3
89bf38087 Delete org.eclipse.jgit.ssh.apache.agent/bin/.project
3e624306d Allow to discover bitmap on disk created after the packfile
e4b95ee56 DfsReader#getObjectSize: use size index if possible
3d4a4bd93 Update jetty to 10.0.20
90674ab15 Update maven plugins
37f64ceea Update org.assertj:assertj-core to 3.25.3
b219040cd Update org.mockito:mockito-core to 5.10.0
1fd8b0f7d Update org.apache.commons:commons-compress to 1.26.0
1f9feb613 Update byte-buddy to 1.14.12
f4f992cdc Add SystemReader.Delegate to reduce boiler-plate code needed to subclass
426d67848 storage.file: Do not throw NSFE when deleting tmp files
67142cacb FS.detectSymlinkSupport: fix a race
99333e75b [errorprone] Fix wrong comparison which always evaluated to false
af5d4d37a [errorprone] Remove unnecessary comparison
617f89131 Errorprone EscapedEntity: Do not use HTML entities inside @code javadoc
e5fd2409c ChangedPathFilter: Suppress warning about backing array (errorprone)
15363e4c3 PackWriterBitmapPreparer: Do not generate bitmaps for excessive branch tips.
e6d83d61e Don't use localized String during JVM shutdown
5afb92aad RawParseUtils.nextLfSkippingSplitLines: fulfil contract as stated
5eff1dd4c [gpg] Refactor the GpgSignatureVerifier
b079968cd RawParseUtils: utility method to get a header value
906c2bebe RebaseCommand: fix stopping on root commit conflicts
c701c01b4 Improve handling of NFS stale handle errors
e0910eda3 Fix handling of missing pack index file
c64664925 CherryPick: add ability to customise cherry-picked commit message
74471b8d7 RawParseUtils: Add missing @since tag for new API method
f94619d49 Silence API warning for new constant
18d0924a4 [ssh] Bump Apache MINA sshd 2.11.0 -> 2.12.0
ba5adc4ce Add tests for handling pack files removal during fetch
ee4443091 PackWriterBitmapPreparer: Set limit on excessive branch count
81f1c0209 Update jna to 5.14.0
307885377 DfsInserter/PackParser: keep min size for index in the inserter
8116f66b9 FooterLines: handle extraction from messages without headers
331fa023e Remove invalid/unnecessary Maven settings
b1cc74b75 Update maven plugins
2199b74ce Update org.eclipse.dash:license-tool-plugin to 1.1.0
ff20da9b8 Enable using slf4j 2.x
80a2b067f Update Tycho to 4.0.4
b450ff180 Update mockito to 5.8.0
3cefca094 Add 4.31 target platform and update orbit to 4.31
a00afad54 Update ecj to 3.36.0
d1189a1cd Renormalize line endings based on .gitattributes
eb0dd3a4e Configure .gitattributes to manage line endings
40e97af2d Normalize line endings to LF for all text files
788487316 BasePackFetchConnection: Skip object/ref lookups if local repo is empty
faa50c683 LooseObjects: Use File#exists when possible
1ab2c73b4 Remove invalid spotbugs configuration
5553f3b30 Fix PMD plugin configuration
b2abab271 DfsReader: give subclasses visiblity over the pack bitmap index
e25bf9573 PackBitmapIndex/StoredBitmap: Expose size and counts
52c18ae15 PackWriter/Statistics: Remove the bitmapt hit stats
a2fed4293 Configure Github issue reporting
ab132937f FooterLine: Protect from ill-formed message
4592603b9 API filter for PackStatistics.objectsWithBitmapsFound
db5ce6b5c StartGenerator: Fix parent rewrite with non-default RevFilter
6a5bde568 Update MANIFEST.MF
b05894436 Document option gc.writeCommitGraph
6cdd04aa4 Bump last release version to 6.8.0.202311291450-r
b54592c8e Prepare 6.9.0-SNAPSHOT builds
8ea5409ec BitmapWalkerTest: New test for the walker
5e563e1ba PackWriter: store the objects with bitmaps in the statistics
555224258 FooterLine: First line cannot be a footer
acf97be2d Reapply "Improve footer parsing to allow multiline footers."
e99fb6edc Revert "Improve footer parsing to allow multiline footers."
796fb2e5c Prepare 6.8.1-SNAPSHOT builds
c43560487 JGit v6.8.0.202311291450-r
6514a46a3 Rename method parameter to fix warning about hidden field
29ad7d56e Fix warnings about empty control flow statements
e4abd0123 BitmapIndexImpl: externalize error message
232813bfc Suppress not-externalized string warnings
ff0ec5c61 Silence API error for new method BitmapIndex#addBitmapLookupListener
5a717a705 Update Orbit to orbit-aggregation/2023-12
ef901e9ae Adapt to type parameter added in commons-compress 1.25.0
340cc787a Improve footer parsing to allow multiline footers.
2a739ad2c Make the tests buildable by bazel test
9a05ca42b Prepare 6.8.0-SNAPSHOT builds
7bc697dfd JGit v6.8.0.202311212206-rc1
aab75dba7 BitmapIndex: Add interface to track bitmaps found (or not)
c0d15c2ab BitmapWalker: Remove BitmapWalkListener
cd8322091 Update repository URLs in CONTRIBUTING.md
823b6a45f Fix typo in FileUtils.isStaleFileHandle() javadoc
e4a341db4 Fix branch ref exist check
52af8dbaf gpg.bc: Supress errorprone InsecureCryptoUsage
5c173c632 Adjust javadoc to pass errorprone checks
705e3d686 Update org.apache.commons:commons-compress to 1.25.0
2986d39ae Fix annotation of deprecated constant CONFIG_KEY_STREAM_FILE_TRESHOLD
e612c2522 BitmapWalkListener: Use plain interface with noop instance
4d82d0aa1 BitmapWalkListener: Add method and rename for commits
8af2f785e Update bouncycastle to 1.77
754a1b492 PatchApplier: wrap output's TemporaryBuffer with a CountingOutputStream
d3f711ca1 BitmapWalker: announce walked objects via listener interface
d9568eda9 Prepare 6.8.0-SNAPSHOT builds
8db605620 Update jetty to 10.0.18
ac0a99597 SSH: bump org.apache.sshd to 2.11.0
375710194 JGit v6.8.0.202311151710-m2
91c914622 Fix typo in constant name CONFIG_KEY_STREAM_FILE_TRESHOLD
4aaf8cad9 Simplify StringUtils#commonPrefix
4f18c5095 Optimize RefDirectory.getRefsByPrefix(String...)
fcea1fe49 CommitGraphWriter: Remove unnecesary progress update call
b761a0fc3 Use try-with-resource to ensure UploadPack is closed
32f7db784 Fix hiding field warning
15c6947fd Fix warning for empty code blocks
3652dd437 Fix boxing warnings
3ca786d76 errorprone: remove unnecessary parentheses
50d670133 Update mockito to 5.7.0 and bytebuddy to 1.14.9
6007371e3 Enable Maven reproducible builds
97afcb050 Upgrade bazlets to the latest revision
7ac2d2573 Revert "Optimise Git protocol v2 `ref-prefix` scanning"
38344badf Document GIT_TRACE_PERFORMANCE to show timings
8e9eab799 config-options.md: fix sort order
ac70632f5 ComboBitset: Add Javadoc
593fbf7c3 CommitGraphWriter: Add progress monitor to bloom filter computation
5207bf070 CommitGraphWriter: Use ProgressMonitor from the OutputStream
c46b54eea CommitGraphWriter: Unnest generation-number progress
3937300f3 Optimise Git protocol v2 `ref-prefix` scanning
1c320d0d4 UploadPackTest: Cover using wanted-refs as advertised set
5f563e386 UploadPack: use want-refs as advertised set in fetch v2
093bde518 BasePackFetchConnection: Avoid full clone with useNegotiationTip
7b2005d52 .gitignore: ignore all Maven output directories `target/`
26d6c325d benchmarks: use org.eclipse.jgit-parent as parent pom
8ca649c16 Generate SBOMs using cyclonedx maven plugin
f91afe5f5 DfsPackFile: Do not attempt to read stream if pack doesn't have it
9323b430b PackObjectSizeIndexLoader: Log wrong bytes on exception
c89f7b874 Silence API warnings for API added in 5.13.3
0f078da4e FileBasedConfig: in-process synchronization for load() and save()
f6774fa8e FileUtils.rename(): better retry handling
cb46ee354 FileBasedConfig: ensure correct snapshot if no file
621685d3c DeleteBranchCommand: update config only at the end
ecf94d159 Config.removeSection() telling whether it changed the config
f93ccb7fd RebaseCommand: return correct status on stash apply conflicts
0b5d4c3aa Use net.i2p.crypto.eddsa 0.3.0 from new Orbit build
5705e2af1 [errorprone] Fix InconsistentCapitalization
5138b97b6 TestRepository: Add getInstant method
790fa7e81 Update orbit to orbit-aggregation/2023-12
e0acf2579 Prepare 6.8.0-SNAPSHOT builds
e5a79343c JGit v6.8.0.202310031045-m1
42917767f UploadPack: Delay freeing refs in sendPack()
39707c673 ByteBufferInputStream: add missing @since 6.8
0ca7438f1 Fix log level for successful execution of ShutdownHook#notify to debug
cf9d05485 Eclipse features: update copyright year
2ab7c3ebe SSH agent: correct plug-in title
b6b1e0a0a Fix DefaultCharset bug pattern flagged by error prone
6672bacca Activate additional error prone checks
a2bce029a WorkingTreeIterator: directly filter input stream
84ced89dc [errorprone] Add missing javadoc summary
d56ae55c8 [errorprone] Fix BadImport
a7edc7889 [errorprone] Prevent int expression may overflow
4cfe27dd0 [errorprone] Suppress ByteBufferBackingArray
ac3794bf0 [errorprone] Suppress JavaLangClash to avoid breaking change
049782743 [errorprone] Fix EqualsGetClass
8c42901b5 [errorprone] MockSystemReader: fix CatchAndPrintStackTrace
34fb4d487 [errorprone] MetaFilter: fix IdentityHashMapUsage
7b79feb07 [errorprone] InternalFetchConnection: Suppress CatchAndPrintStackTrace
f8d91c273 [errorprone] Don't swallow exception
298f93e98 [errorprone] Fix inconsistent capitalization
e5e54b61b [errorprone] PageRef#isStringRef: suppress UnusedMethod
37f60c798 [errorprone] FileReftableStack: fix EqualsUnsafeCast
bf92bb9bc [errorprone] Suppress MissingSummary for translation bundles
e23796cbd [errorprone] CommandLineMergeTool: Remove unused constuctor
254ffbfb0 [errorprone] SimilarityRenameDetector: suppress IntLongMath
7092803ad Add comment why protocol list uses WeakReference
d5bcf199c [errorprone] Transport: Suppress ModifyCollectionInEnhancedForLoop
8302377d9 [errorprone] Rename method parameter to silence InvalidParam
e6c808600 [errorprone] MyersDiff#main: fix CatchAndPrintStackTrace
2d16df1a0 [errorprone] TagCommand: Remove unused parameter of private method
28d4b3490 [errorprone] PathFilter#getPathsBestEffort: fix ArrayAsKeyOfSetOrMap
cda8ffdbb [errorprone] DirCacheEntry: suppress JavaInstantGetSecondsGetNano
d0b9ab1c4 [errorprone] Directly implement functional interface
ba2f9affc [errorprone] InterruptTimer#terminate: ensure Thread#join succeeds
a17e05437 [errorprone] RawParseUtils@parseHexInt64: suppress IntLongMath
978cbb84c [errorprone] Exceptions should not override #toString
1e92426c5 [errorprone] Fix invalid param in javadoc
e53c1864e [errorprone] FS#searchPath: handle surprising behavior of String#split
a94e54ce8 [errorprone] FileStoreAttributes#FUTURE_RUNNER: Fix corePoolSize
13bea495c [errorprone] Express duration using seconds which is clearer
ec0657ba0 [errorprone] Use ArrayList instead of LinkedList
b4f4ae472 [errorprone] AddCommand#filepattern: use a more specific type
ebdf71c11 [errorprone] Suppress UnnecessaryParentheses
4cf246c9a [errorprone] Remove unnecessary parentheses
d65170603 tools/BUILD: Sort errorprone rules alphabetically
136b8953e Update jetty to 10.0.16
ff10cd2d8 Update bazel to 6.3.2
01d6d8a19 DfsPackFile: remove unnecessary @SuppressWarnings("boxing")
1924d353e CommitGraphWriter: Remove unnecessary semicolon
4262150f7 CommitGraphWriter: fix boxing warnings
c57257bc0 CommitGraphWriter#createCoreChunks: Remove not-thrown exceptions
2390a89f2 CommitGraphWriter: Decouple Stats from computing bloom filters
f1a9d92a3 CommitGraphWriter: Move bloom filter calculation out of createChunks
e0bd4882f Documentation: Move writeChangedPaths flag from commitGraph to gc section.
916200e27 [errorprone] Fix wrong comparison which always evaluated to false
cf5ec856b [errorprone] Remove unnecessary comparison
290902d9f Update org.eclipse.jdt:ecj to 3.35.0
12e9c3f1f Update org.apache.commons:commons-compress to 1.24.0
0acc5aeaf [errorprone] Suppress ImmutableEnumChecker for ShutdownHook
8d62e5d04 Update bytebuddy to 1.14.8
642f16023 Use ShutdownHook to gracefully handle JVM shutdown
d4d6c2b5a Add ShutdownHook to cleanup FileLocks on graceful JVM shutdown
f94be665f Unregister ShutdownHook when GC#PidLock is closed
e3798df6e OSGi: move plugin localization to subdirectory
3bc1da30d ssh: Remove redundant null check for home.getAbsoluteFile()
82c6638c7 RevertCommand: support for inserting a Gerrit change ID
7e502953c [releng] Bump japicmp base version to 6.7.0.202309050840-r
7baa5a157 DfsPackFile: Record index loads only in one place
30427485d DfsPackfile: Emit the index load with the index object
3a3b72cf9 Prepare 6.8.0-SNAPSHOT builds
bb12dd4cb Prepare 6.7.1-SNAPSHOT builds
09f6383f5 Switch back to wagon-ssh-external
b43880484 Fix generation of maven site
ba1653162 Ensure home directory not null before using in Sshdsessionfactory
4f5afe9d7 CommitGraphWriter: Make the list of chunks immutable
f90f0717a CommitGraphWriter: Assert written bytes
ea02caf1e JGit v6.7.0.202309050840-r
c024cb23d Remove unused API problem filters
a5ffa44d5 [releng] Bump japicmp base version to 6.6.0.202305301015-r
43681ee74 Fix list of 3rd party bundles in p2 repo
ee0b20c64 Add missing source bundle org.osgi.service.cm.source to target platform
13c8dacae CommitGraphWriter: throw exception on unknown chunk
fb51a2234 Document commit-graph options supported by JGit
a40abb0a1 Fix warning raised for local variable hiding DfsPackFile#index
4f3db912c Suppress boxing warnings in DfsPackFile
807cf678a Remove unused API problem filters
02b45a844 Suppress boxing warnings in tests
35b5344c9 Fix warning about empty block
e51a0e51e Update mockito to 5.5.0
24c5d0120 Update byte-buddy to 1.14.7
c5d8936c8 Prepare 6.7.0-SNAPSHOT builds
c54acc582 JGit v6.7.0.202308301100-rc1
2be8bf2b3 Remove the cbi-snapshots Maven repository
8995a6429 Update Orbit to orbit-aggregation/release/4.29.0
2a64412e9 Add target platform for Eclipse 2023-09 (4.29)
9fd44e098 Use release p2 repo for Eclipse 2023-06 (4.28)
2d0a5fa64 Update tycho to 4.0.2
b6b0d323a Update jmh to 1.37
46264ba48 Update bouncycastle to 1.76
e29834a77 Fix some tests in ConfigTest
a2f326b76 Handle global git config $XDG_CONFIG_HOME/git/config
0d5e01761 IO: use JDK convenience methods
055a7c20e org.eclipse.jgit.junit.ssh/.settings/.api_filters: fix unclosed tags
a56930723 ReadChangedPathFilter: fix Non-externalized string literal warning
e7a09e316 Introduce core.packedIndexGitUseStrongRefs config key
9919a9faa DfsReader: Make PackLoadListener interface visible to subclasses
551ca93cc DfsGarbageCollector: provide commit graph stats
6f7333693 DfsGarbageCollector: put only GC commits into the commit graph
b4b8f05ee DfsReader: Expose when indices are loaded
de7b5b7b2 Prepare 6.7.0-SNAPSHOT builds
1d26471c1 JGit v6.7.0.202308011830-m2
24b6a35d3 Add missing @since tags
ba8f3cb1f Fix errorprone warning about precedence
c353645a0 Move footer-line parsing methods from RevCommit to FooterLine
462c57ec8 Merge: Add diff3 style merge conflict formatter.
c8f5a3f99 RevCommitCG: Read changed-path-filters directly from commit graph
eecd93714 Update commons-codec to 1.16.0
68c08265a Add missing @since tags for new API methods
8879eec46 Add missing package import needed to use MurmurHash3
ec3d919aa Identify a commit that generates a diffEntry on a rename Event.
f196c7a0e Pack: open reverse index from file if present
000e7caf5 PackReverseIndexV1: reverse index parsed from version 1 file
2eba4e5b4 PackReverseIndex: open file if present otherwise compute
8123dcd69 PackReverseIndex: verify checksums
7d2669587 ComputedPackReverseIndex: Clarify custom bucket sort algorithm
3b77e33ad CommitGraphWriter: add option for writing/using bloom filters
77aec6214 CommitGraphWriter: reuse changed path filters
d3b40e72a RevWalk: use changed path filters
ff0f7c174 CommitGraphLoader: read changed-path filters
49beb5ae5 CommitGraphWriter: write changed-path filters
23758d7a6 ssh: PKCS#11 support
db08835c6 GC: Remove handling of extra pack for RefTree
f41c5e8c0 Remove unnecessary @SuppressWarnings("serial")
010a14f24 Remove unused API problem filters
b2f7dc189 Remove redundant specification of type arguments
760bdd09b DfsPackParser: Create object indices if config says so
afb013b98 PackParserTest: Extract pack-writing helper code to its own class
cb99ff5bb DfsInserter: generate object size index if config says so
4d2a003b9 DfsInserter: populate full size on object insertion
12a4a4cca DFSGarbargeCollector: Write object size indices
9dace2e6d DfsReader/PackFile: Implement isNotLargerThan using the obj size idx
8a053b57a [releng] Use tycho 4.0.0 instead of 4.0.0-SNAPSHOT
dceebe350 Update mockito to 5.4.0 and bytebuddy to 1.14.5
79b46a0ef Fix S3Repository getSize to handle larger object sizes
91b23cc55 DfsPackFile: make #getReverseIdx public
55230e2b9 Add missing bazel dependency to benchmarks
4d5f2ada5 Update JMH to 1.36
00fa17680 Update bouncycastle to 1.75
d2f114589 Update org.eclipse.jdt:ecj to 3.34.0
8f1693c2d Update eclipse-jarsigner-plugin to 1.4.2
23a0e7e06 Update maven-surefire-report-plugin to 3.0.0
6eb5a4cc8 Update maven-project-info-reports-plugin to 3.4.3
9a60f7f7a Update maven-compiler-plugin to 3.11.0
5fcd681ac Update maven-enforcer-plugin to 3.3.0
6250020fa Update build-helper-maven-plugin to 3.4.0
a00f1663d Update spring-boot-maven-plugin to 2.7.13
d32352a44 Update maven-source-plugin to 3.3.0
8e6197162 PackReverseIndex: separate out the computed implementation
faefa90f9 Default for global (user) git ignore file
7b955048e Fix all Javadoc warnings and fail on them
c7960910f Mark COMMIT_GENERATION_* constants final
74547f4a6 PackReverseIndex: use static builder instead of constructor
181b629f7 Gc#writePack: write the reverse index file to disk
0f071a7bc Bump bazel vesion to 6.2.0
9afff3e80 Prepare 6.7.0-SNAPSHOT builds
Release-Notes: Update JGit to c0b415fb0
Change-Id: I961bdb25823fcd03954214ada46ca24d76ee2fa5
(cherry picked from commit eef3529120308bbbd4a6b309f3c1f00bd724a9cd)
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
While work-in-progress changes are unsubmittable, they can be
mergeable. This is important if a site has set
change.mergeabilityComputationBehavior to a value other than NEVER.
Without this change, WIP changes always show as having a merge conflict
when viewed as search/query results. A new test is added for that
behavior and it fails prior to this change.
Bug: Issue 40014889
Change-Id: I8ae13f6563fd7a527b1af92ba1533433949b61f9
Release-Notes: Fixed WIP changes always showing Merge Conflict in searches
(cherry picked from commit e48a495f9d4da5fcd49f10ce756538c19e0deeec)
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add a new test for this behavior to show that the child change does not
cause a regression or any change in behavior for submit.
Release-Notes: skip
Change-Id: I4b45699ceccbdc3cf71e007ad8487f11f8304972
(cherry picked from commit 07e2deb03bc15959959c036bad30e418c094508f)
|
| |_|_|_|/
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
So that reviewed flags are removed from the correct change.
Release-Notes: Ensure that clearing reviewed flags is performed on correct change when it is imported from another instance
Forward-Compatible: checked
Change-Id: I27899f76f804d41eacdf9a0f81fc74c012ff2c03
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Introduce `virtualId` to change data and initiate it with the
`legacy_id_str` from the index so that changes list screen (which is
populated from the index only) could be properly updated with star bit.
Ensure that `changeServerId` is loaded from notes when notes are
available in the `ChangeData`.
Finally use the `virtualId` in all calls to get/set starred changes so
that the refs clash is avoided. As a result the starred changes ref
gets modified to the following form:
refs/starred-changes/NN/<virtual-id>/<account-id>
where `virtual-id` is `server-id` encoded `change-id` for imported
changes and `change-id` for existing/newly created ones.
Issue 331016670: to be addressed as the follow-up - toggle star from
changes list screen doesn't work for imported changes - the problem is
that UI uses change.id to obtain the project first which clashes with
existing changes. It works fine on the single change screen.
Bug: Issue 325309573
Release-Notes: Starred changes clash when importing changes from another instance
Forward-Compatible: checked
Change-Id: I85c946efcab5d20dd25682cb758bb9601c93b1d2
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit addresses ambiguity in draft comment references when dealing
with imported changes. Previously, the system encountered issues when
two changes shared the same change number, with one being imported. The
solution introduces a new pattern for creating draft comment references:
refs/draft-comments/NN/<virtual-id>/<account-id>.
This approach ensures distinct draft comment references for each
imported change, eliminating the side effects caused by sharing the
same reference with non-imported changes.
Bug: Issue 3253095744
Forward-Compatible: checked
Release-Notes: Resolve draft comment reference ambiguity for imported changes
Change-Id: I108671395e047f688744e1c4a7ed3bee8d359735
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Details: https://errorprone.info/bugpattern/DereferenceWithNullBranch
Release-Notes: skip
Change-Id: Id58ac064f86377ad27ea494e78f90c3355ef110c
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The Change class is created based on index lookup and subsequent
load from NoteDb. Before this change, even though the serverId information
was potentially available, it was not copied to the in-memory object
making more difficult for the other parts of the Gerrit code to understand
if the change was created locally or imported from a different server.
Copy the serverId to the Change fields so that it become easier and
clearer to understand where the change was initially coming from.
Release-Notes: skip
Forward-Compatible: checked
Change-Id: Ic0e2cf4d8e77a6c3f9a64e2572698b48e3dd90e1
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
When calling the virtualisation algorithm for change numbers on the
local serverIds, make the function a NOOP and return the same number.
This avoids code duplication of the logic in different parts of Gerrit
and eliminates the need of knowing how the algorithm works from outside
the calls, treating the virtualisation as a black-box function.
Release-Notes: skip
Change-Id: I385dcaf5e40e8429e9981040a73cde3a4214c4e4
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Release-Notes: skip
Change-Id: I716b81fb22d68b967a15236460ce6847bed32256
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Release-Notes: skip
Change-Id: Ie4c37188394c076079ab8127aea0ac206882030c
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We won't be fixing these warnings on stable branches, so disable them to
quiet the build output.
Release-Notes: skip
Change-Id: I48eeee20541ad2240a3fe5cf3db797f34801f23c
|
|\ \ \ \ |
|
| |\| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* stable-3.6:
Consistently normalize PatchSetApprovals when persisting
Change-Id: I32d045829fcbae740f834ad6967e3f76c7ac9c44
Release-Notes: skip
|
| | |\| |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* stable-3.5:
Consistently normalize PatchSetApprovals when persisting
Nothing to merge since this method is implemented differently in 3.6+
where approvals are only persisted and not inferred.
Release-Notes: skip
Change-Id: I3c2f820a565c35eead7f07e3dcf1ec1111841840
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Comparing an unnormalized psa with a normalized psa can create
unexpected differences, leading to error messages during the
copy-approvals command like:
java.lang.IllegalArgumentException: Approval that should be copied is not copied.
Fix those situations by applying normalization to the current approvals
before comparing them to the inferred approvals (which already have
normalization applied).
Release-Notes: Fix copy-approvals for cases where current approvals have different values when normalized
Change-Id: I17cccf537943654f89a4d02af13ec2dda768f0df
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Change I36c1e2f56127a7e5a11769bf36daf586e32d8bcf excluded drafts and
stars from the ref_state index field but failed to exclude the related
patterns from the ref_state_pattern. As a result, any change that had a
draft comment or a star was always stale, reindexing such a change would
never get it out of the stale state.
Release-Notes: Fix change staleness check for changes with drafts and stars
Forward-Compatible: checked
Change-Id: Ie66138180f3df65c82624905ecdab915992b381b
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Update plugins/delete-project from branch 'stable-3.7'
to 8f6ea7eeb22f8b8487d3b433d367272172ea32d1
- Add support for newer plugin-node-resolve versions
Due to [0] in plugin-node-resolve, it changes the way modules are
resolved. It now looks at package.json and figures it out from
there although if the stuff it is looking for is not found
it falls back to how it did the resolving in older versions.
Unfortunately there is no config to force the behaviour
so we have to use the new behaviour.
This is what it looks like:
```
"exports": {
".": {
"development": "./development/reactive-element.js",
"default": "./reactive-element.js"
},
....
"./decorators.js": {
"development": "./development/decorators.js",
"default": "./decorators.js"
},
....
},
```
Thus "./decorators" fails to match "./decorators.js" and so it fails
to resolve.
[0] https://github.com/rollup/plugins/commit/3d60158f21e0b6a5a8a53d9977928e2e148cb885
Change-Id: I7829591e45662ee6028182beb57396b327823275
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Release-Notes: skip
Forward-Compatible: checked
Change-Id: I39cd5539e34b80902d6fa246d740d974ed7ec50c
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Re-apply module update performed in Id1f96b09c98
and lost in the merge-up of stable-3.6 into stable-3.7
in I169d7c1ded.
Release-Notes: Update JGit to acf21c0bc which was missed in v3.7.7
Change-Id: I9c43f8262cdec5757e0ce695870595115d9a5bc0
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The Change I34dbf53524 introduced a specific flag for parsing
the change id in SSH commands differently for some use-cases
where there is a requirement of NOT using a Lucene index lookup.
When the flag was introduced in v2.13, Gerrit had a radically
different architecture:
- The Git repository contained only the code
- All review metadata was stored in ReviewDb
- The primary key for looking up changes was the change number
which did not require any Lucene lookups
At that time in v2.13, the introduction of the useIndex option
allowed to fix a specific SSH command for reindexing a change
in case it was missed during the change creation of patch-set
upload.
See Issue 313935024 for more details on how a Gerrit change
may end up with a stale or completely absent indexing entry.
Starting from v2.15 and then up to v3.0, the removal of
ReviewDb code-base was implemented without paying too much
attention on the consequences on the useIndex flag which was
ported "syntatically" to the newer releases but lost completely
its effectiveness. Also, the SSH commands did not have historically
enough test coverage, exposing the risks of a regression.
On the other side, the change finder is now clever enough
to use the index or the underlying repository depending on
the format of the change id, which makes a lot more sense
than a static decision based on the useIndex flag.
Also remove the unneeded reload of the change notes found and
loaded by the changeFinder, which was inadvertently added as part
of Change Idaeeb7d82.
TODO: Remove the flag completely after this change is
merged to master. The flag is kept unused in stable
versions for not invalidating the public signature
of the methods, which may be used by other parts of
Gerrit or other plugins.
Bug: Issue 325821304
Release-Notes: Fix change id parsing in SSH commands
Change-Id: Iee2057f8978a40d37804c9198df0c1c70fdcb9eb
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Release-Notes: skip
Change-Id: I3cda0382fa299c7724d03820800abc51dc8f3289
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Release-Notes: skip
Change-Id: I8cd566b4369f478f23400b38c3dc40c87b089df2
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* stable-3.6:
Update JGit to acf21c0bc
Release-Notes: skip
Change-Id: I169d7c1ded2db55c678b2512577ae39a088bd569
|
| |\| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* stable-3.5:
Update JGit to acf21c0bc
Release-Notes: skip
Change-Id: I76ac85855496007268d64dfd0d705addd78003b0
|