mistercrunch commented on a change in pull request #4587: [logs] Dropping dt column URL: https://github.com/apache/incubator-superset/pull/4587#discussion_r184273398
########## File path: superset/migrations/versions/30bb17c0dc76_.py ########## @@ -0,0 +1,26 @@ +"""empty message + +Revision ID: 30bb17c0dc76 +Revises: f231d82b9b26 +Create Date: 2018-04-08 07:34:12.149910 + +""" + +# revision identifiers, used by Alembic. +revision = '30bb17c0dc76' +down_revision = 'f231d82b9b26' + +from datetime import date + +from alembic import op +import sqlalchemy as sa + + +def upgrade(): + with op.batch_alter_table('logs') as batch_op: + batch_op.drop_column('dt') Review comment: Not sure, maybe we could have just left the column there as NULL moving forward and add a note about it being deprecated. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services