{"push_id":1081817093,"size":20,"distinct_size":20,"ref":"refs/heads/upstream/master","head":"825ba7e64554c86613f371bbad9ff7d7efe2c2f6","before":"102f8793a7cda9e6d2b2cfcbea78ab2fdd2b0dc7","commits":[{"sha":"680495c014f57d44090fb62cd52e6fc5b4855073","author":{"email":"d43fb6c98c97e553b671ed9a9318100f1de386e7@google.com","name":"Edwin Kempin"},"message":"Store hashtags in change index\n\nWhen changes are queried the hashtags are retured with each change.\nThese hashtags should be read from the change index rather than\nloading them from NoteDb. Loading the change notes for each change in\nthe result set leads to poor query performance when NoteDb is enabled.\n\nThe existing HASHTAG field in the change index contains the lower case\nhashtags to support case-insensitive queries by hashtag. To be able to\nretrieve the hashtags with the original case from the index we must\nadd a new field that stores the hashtags with the original case. This\nnew field is not used for serving queries, hence it is of type\nSTORED_ONLY.\n\nChange-Id: I0a9eb0ad0ac849d8b7b3ac6965dc8b5f38d04a32\nSigned-off-by: Edwin Kempin <ekempin@google.com>","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/680495c014f57d44090fb62cd52e6fc5b4855073"},{"sha":"af718612804947936d7bfa1ab08d6b653ead3374","author":{"email":"ac39666f39dfa5b37bad5e19ae3f186528808854@google.com","name":"Dave Borowitz"},"message":"Support disabling NoteDb reads for tests\n\nIn I0a9eb0ad we fixed a NoteDb-specific issue where loading the\ndefault search results page caused ChangeNotes to get loaded, despite\nnot causing changes to load from ReviewDb.\n\nSupport this configuration in tests by adding a method to\nNotesMigration that is called in AbstractChangeNotes#load(). In\nproduction this method always returns false so the cost is negligible.\n\nChange-Id: I22622cb3c834eeb1391d5b80247beafe36fffedf","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/af718612804947936d7bfa1ab08d6b653ead3374"},{"sha":"387e3a42572c28e81c693c30cc401e16fb2284f5","author":{"email":"682ad9f487f614f19e8b7cc5d340bdc8e1c1b128@chromium.org","name":"Andrew Bonventre"},"message":"Add getPluginName to Plugin prototype\n\nAlso add a warning for Gerrit.getPluginName since it relies on\nbehavior unsupported in strict mode (arguments.callee).\n\nBug: Issue 3915\nChange-Id: Ie0789ab14b140c08b7e5be26c1e14c9136b15d9b","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/387e3a42572c28e81c693c30cc401e16fb2284f5"},{"sha":"be351a65e3551ac512a57b6f9053e6c00a21f088","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"Remove additional leftover classes and CSS styles from the old UI\n\nThis removes additional unused classes and CSS styles from the old\nchange screen and diff screen.\n\nChange-Id: I6825f8c810f4dd12ec95b762fb8d2bc8302f3ed2","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/be351a65e3551ac512a57b6f9053e6c00a21f088"},{"sha":"45344676687b02d80dd711f5e67d849ff8e07391","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"EditPreferencesBox: Simplify initKeyMapType()\n\nLoop over KeyMapType.values() instead of explicitly listing them.\n\nChange-Id: I53dd3372c2860bf2a565485fa8f27438361af1bc","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/45344676687b02d80dd711f5e67d849ff8e07391"},{"sha":"72096e0bbbf4ca595c5625c715e118adf3ec91dc","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"Simplify getNativeEvent().getKeyCode() to getNativeKeyCode()\n\nFor KeyDownEvent and KeyUpEvent, the latter is just a shorthand for the\nformer. Unfortunately, KeyPressEvent doesn't have this shorthand.\n\nChange-Id: I6dc37b93cf09be5cfa58bbb5fcc81d7f4b9a09e5","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/72096e0bbbf4ca595c5625c715e118adf3ec91dc"},{"sha":"4cb4f0a57c5674997518195a6c6993c5aadba56d","author":{"email":"0435ee366cc8d9c78e1b95b424c78c0437d6ddc1@gmail.com","name":"David Ostrovsky"},"message":"Merge \"EditPreferencesBox: Simplify initKeyMapType()\"","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/4cb4f0a57c5674997518195a6c6993c5aadba56d"},{"sha":"90f29bd05f5171ad023a077e67419287dd56c462","author":{"email":"5fce32887d235858b7e7ca81f4a8e27f52e1c330@fwo.ch","name":"Urs Wolfer"},"message":"Groups-API: Move NotImplemented class to correct place\n\nFollow-up fix for Ieb1dd15ec. Inner-class was inside inner-class\n'ListRequest' instead of 'Groups' directly.\n\nChange-Id: Ie6ef70c22fc716373a956db6d9798a18a8945353","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/90f29bd05f5171ad023a077e67419287dd56c462"},{"sha":"ae5a24759b29bb00289781506a80a7e9031f9b85","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"DiffScreen: Remove duplication in handling skipped regions\n\nIn the change that added the new unified diff screen [1],\nUnifiedSkipBar / SideBySideSkipBar and UnifiedSkipManager /\nSideBySideSkipManager are almost identical. Extract common code to\nremove a lot of duplication.\n\nAdd isSideBySide() to DiffScreen to make it easier to distinguish the\ntwo view types.\n\nAlso, refresh the CodeMirror instance when a SkipBar changes. This\nupdates the scrollbar overview ruler to reflect the new displayed\nregion when a skipped line changes.\n\n[1]: https://gerrit-review.googlesource.com/#/c/49171/\n\nBug: Issue 4069\nChange-Id: I3d66812e28c4abe7ba70e30d42026e80e14ee35d","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/ae5a24759b29bb00289781506a80a7e9031f9b85"},{"sha":"4fb61508db495d3ebbc18b0f9388551e7942586e","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"Remove unused constants from ChangeConstants and ChangeMessages\n\nThis removes additional unused constants from the old change screen and\ndiff screen.\n\nChange-Id: I5f36bff55c230f36eac01c7b6a9cc9485a2d9ac2","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/4fb61508db495d3ebbc18b0f9388551e7942586e"},{"sha":"007a21aa229ef5a86c08de48973b66325bdb97bc","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"Make DiffChunkInfo implement Comparable to simplify code\n\nThe start line on the other side of a DiffChunkInfo is always known at\nchunk creation. Pass it to the constructor and make DiffChunkInfo\nimplement Comparable. This both simplifies code and avoids unnecessary\nLineMapper lookups.\n\nChange the fields in DiffChunkInfo to be final and remove the getters.\nThis simplifies code a bit more.\n\nChange-Id: I556c94f9c2a68dda5b81bf303f42f73d981dc465","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/007a21aa229ef5a86c08de48973b66325bdb97bc"},{"sha":"4909996481831eb48fadc2974a1a1f5dd8d1738b","author":{"email":"0435ee366cc8d9c78e1b95b424c78c0437d6ddc1@gmail.com","name":"David Ostrovsky"},"message":"Merge \"Make DiffChunkInfo implement Comparable to simplify code\"","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/4909996481831eb48fadc2974a1a1f5dd8d1738b"},{"sha":"77f3edcec30fbb37971d4e6bbd4fd5c9a9c8870a","author":{"email":"e690c388aec5b811038fa4c17268e9b7582a1bee@gmail.com","name":"Michael Zhou"},"message":"ChunkManager: Make \"lineMapper\" and \"scrollbar\" package private\n\nThis removes the getters and simplifies code. Now that the getter is\nremoved, rename the variable \"mapper\" to \"lineMapper\" to make it more\nreadable.\n\nChange-Id: I33b0db8a25f9aec98517b8201af95eac3bbf0cf3","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/77f3edcec30fbb37971d4e6bbd4fd5c9a9c8870a"},{"sha":"c0650a3f0fbbb56b64b83eb0952fbf6c4b2e18cb","author":{"email":"d43fb6c98c97e553b671ed9a9318100f1de386e7@google.com","name":"Edwin Kempin"},"message":"Merge \"Groups-API: Move NotImplemented class to correct place\"","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/c0650a3f0fbbb56b64b83eb0952fbf6c4b2e18cb"},{"sha":"264d01069b66d9448fa45191c2bb6cdb09fa896e","author":{"email":"3fc1c5732e6ee2d6b84ec340e9112d1b98672960@gmail.com","name":"Doug Kelly"},"message":"Expose change index endpoint to Extensions API\n\nThe index endpoint could be useful to extensions as a way to\nprogramatically reindex a change.  This isn't meant as an excuse for\nmanipulating data outside of Gerrit, but is more intended for\nmaintenance type plugins which may be used to check the consistency\nof Gerrit's internal state, such as healthcheck.\n\nChange-Id: I34ce7959c0c53e0e7b2b4ff7cd92cdc29b893770","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/264d01069b66d9448fa45191c2bb6cdb09fa896e"},{"sha":"84cd28a0635e403c2f31978c6feee6513b02328a","author":{"email":"b162584c9d065018308db99337b5a9fbc8680008@sonymobile.com","name":"David Pursehouse"},"message":"ssh/GarbageCollectionIT: Add more explicit assertions for error case\n\nAssert that the session has an error, and that the error message is not\nnull.\n\nThis causes the test to fail early in the case that the expected error\nwas not raised, whereas before it would fail with NPE when dereferencing\nnull returned by getError().\n\nChange-Id: Ie1e30439ccd642660bdfa87f0b43e9f56164688f","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/84cd28a0635e403c2f31978c6feee6513b02328a"},{"sha":"8f02dff77f7935d14fff6f46116ae344333b3a2f","author":{"email":"b162584c9d065018308db99337b5a9fbc8680008@sonymobile.com","name":"David Pursehouse"},"message":"AbstractDaemonTest: Add user SSH session\n\nRename the adminSession and userSession members to adminRestSession\nand userRestSession.\n\nRename sshSession to adminSshSession.  Add userSshSession.\n\nChange-Id: Id705a04317d146cea48b6215e78d8f8ea7fa95a1","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/8f02dff77f7935d14fff6f46116ae344333b3a2f"},{"sha":"8d3054c6cd770d70f19ad189f3b01a211cf40bed","author":{"email":"d43fb6c98c97e553b671ed9a9318100f1de386e7@google.com","name":"Edwin Kempin"},"message":"Merge \"ssh/GarbageCollectionIT: Add more explicit assertions for error case\"","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/8d3054c6cd770d70f19ad189f3b01a211cf40bed"},{"sha":"417072beae316b234a8f8707913c5755aa7cee18","author":{"email":"d43fb6c98c97e553b671ed9a9318100f1de386e7@google.com","name":"Edwin Kempin"},"message":"Merge \"AbstractDaemonTest: Add user SSH session\"","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/417072beae316b234a8f8707913c5755aa7cee18"},{"sha":"825ba7e64554c86613f371bbad9ff7d7efe2c2f6","author":{"email":"d43fb6c98c97e553b671ed9a9318100f1de386e7@google.com","name":"Edwin Kempin"},"message":"Merge \"Expose change index endpoint to Extensions API\"","distinct":true,"url":"https://api.github.com/repos/openstack-infra/gerrit/commits/825ba7e64554c86613f371bbad9ff7d7efe2c2f6"}]}