New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 617703 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Update tko_test_attributes table to use BIGINT for id

Project Member Reported by dshi@chromium.org, Jun 6 2016

Issue description

The table schema needs to be updated so id field is a BIGINT to avoid reaching the max value. test_attributes is a table tends to have lots of data, tests can dump any keyval in that table. So we need an id with bigger range.

 
Owner: dshi@chromium.org
Status: Assigned (was: Available)

Comment 2 by dshi@chromium.org, Feb 2 2017

Labels: OS-Chrome
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/d8220fa92c8777d84f51f0eed55ae7827a6120c4

commit d8220fa92c8777d84f51f0eed55ae7827a6120c4
Author: Dan Shi <dshi@google.com>
Date: Fri Feb 03 13:15:07 2017

[autotest] Update TKO db schema

1. Add index for tko_tests.finished_time
2. Update tko_test_attributes id key to bigint.

BUG= chromium:617703 
TEST=local run migrate.py sync

Change-Id: I9167f2fcea2a99ac2bf6e9757e1d05b1fbe67104
Reviewed-on: https://chromium-review.googlesource.com/436485
Commit-Ready: Dan Shi <dshi@google.com>
Tested-by: Dan Shi <dshi@google.com>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[add] https://crrev.com/d8220fa92c8777d84f51f0eed55ae7827a6120c4/frontend/migrations/108_index_tko_tests_finished_time.py
[add] https://crrev.com/d8220fa92c8777d84f51f0eed55ae7827a6120c4/frontend/migrations/109_change_tko_test_attributes_id_data_type.py

Comment 4 by dshi@chromium.org, Feb 21 2017

Status: Fixed (was: Assigned)

Comment 5 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 6 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 7 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 8 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment