Issue metadata
Sign in to add a comment
|
master-paladin: SQL error in CommitQueueHandleChanges if all builds are cancelled? |
||||||||||||||||||||||||
Issue descriptionhttps://cros-goldeneye.corp.google.com/chromeos/healthmonitoring/buildDetails?id=2874098 CommitQueueHandleChanges failed ProgrammingError: (ProgrammingError) (1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1") 'SELECT * from hwTestResultTable WHERE build_id IN ()' ()
,
Aug 24
Can I take this issue if there is no one working on this?
,
Aug 24
Sure!
,
Sep 8
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/chromite/+/6c8c214ab77ea82ce403db49fba20a719b936ea8 commit 6c8c214ab77ea82ce403db49fba20a719b936ea8 Author: Jongheon Kim <sapzape@gmail.com> Date: Sat Sep 08 10:23:00 2018 Add syntax error exception for empty values This patch is add exception for empty value when sql execute in GetHWTestResultsForBuilds Bug: 877342 Change-Id: Ibd9a9c04bbefb55f840a5299e3d8bd8d523d787e Reviewed-on: https://chromium-review.googlesource.com/1190383 Commit-Ready: Jongheon Kim <sapzape@gmail.com> Tested-by: Jongheon Kim <sapzape@gmail.com> Reviewed-by: Mike Frysinger <vapier@chromium.org> [modify] https://crrev.com/6c8c214ab77ea82ce403db49fba20a719b936ea8/lib/cidb.py [modify] https://crrev.com/6c8c214ab77ea82ce403db49fba20a719b936ea8/lib/cidb_integration_test.py
,
Oct 1
IIUC, this is still happening, and people are actively working on it on bug 890651 . |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by jclinton@chromium.org
, Aug 24Status: Available (was: Untriaged)