[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-06-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Bartosz Dziewoński matma@gmail.com changed:

   What|Removed |Added

 Blocks||67151

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-05-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

--- Comment #15 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 125769 merged by jenkins-bot:
Drop rc_cur_time from recentchanges

https://gerrit.wikimedia.org/r/125769

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-05-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Umherirrender umherirrender_de...@web.de changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED
   Assignee|wikibugs-l@lists.wikimedia. |umherirrender_de...@web.de
   |org |
   Target Milestone|Future release  |1.24.0 release

--- Comment #16 from Umherirrender umherirrender_de...@web.de ---
Field is now dropped in master

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-05-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

--- Comment #13 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 131852 had a related patch set uploaded by EBernhardson:
Stop populating rc_cur_time

https://gerrit.wikimedia.org/r/131852

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-05-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

--- Comment #14 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 131852 merged by jenkins-bot:
Stop populating rc_cur_time

https://gerrit.wikimedia.org/r/131852

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-04-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Umherirrender umherirrender_de...@web.de changed:

   What|Removed |Added

   See Also|https://bugzilla.wikimedia. |
   |org/show_bug.cgi?id=58272   |

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-04-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

--- Comment #12 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 125769 had a related patch set uploaded by Umherirrender:
Drop rc_cur_time from recentchanges

https://gerrit.wikimedia.org/r/125769

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2014-04-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2013-12-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Marcin Cieślak marcin.cies...@gmail.com changed:

   What|Removed |Added

   See Also||https://bugzilla.wikimedia.
   ||org/show_bug.cgi?id=58272

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2013-12-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

--- Comment #11 from Umherirrender umherirrender_de...@web.de ---
This way was used to avoid the third party updaters to restore his database,
when he have to downgrade (or roll back the upgrade).

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2013-12-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Marcin Cieślak marcin.cies...@gmail.com changed:

   What|Removed |Added

 CC||afeld...@wikimedia.org,
   ||m...@everybody.org

--- Comment #10 from Marcin Cieślak marcin.cies...@gmail.com ---
To reiterate what I said in the code review for gerrit change 99068:

I think that the commit should remove the field from both SQL as well as all
table definitions and add updater definitions for three major databases as you
did in the original gerrit change. WMF is using neither installer nor updater
so I believe they can live with it.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2013-12-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Marcin Cieślak marcin.cies...@gmail.com changed:

   What|Removed |Added

 CC||marcin.cies...@gmail.com

--- Comment #8 from Marcin Cieślak marcin.cies...@gmail.com ---
I am now getting 

Query: INSERT INTO recentchanges
(rc_timestamp,rc_namespace,rc_title,rc_type,rc_source,rc_minor,rc_cur_id,rc_user,rc_user_text,rc_comment,rc_this_oldid,rc_last_oldid,rc_bot,rc_ip,rc_patrolled,rc_new,rc_old_len,rc_new_len,rc_deleted,rc_logid,rc_log_type,rc_log_action,rc_params,rc_id)
VALUES ('2013-12-04 10:54:26
GMT','0','Main_Page','1','mw.new','0','1','0','MediaWiki
default',,'1','0','0','127.0.0.1','0','1','0','524','0','0',NULL,,,'1')
Function: RecentChange::save
Error: 23502 ERROR:  null value in column rc_cur_time violates not-null
constraint

when installing git master (now at 0a54e70cdd2162be4b331aacb79ad5a3926dd2c6) on
PostgreSQL.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2013-12-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

--- Comment #9 from Marcin Cieślak marcin.cies...@gmail.com ---
Submitted gerrit change 99068 to revert this for now.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 40667] Drop unused rc_cur_time field from recentchanges table in version 1.24

2013-11-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=40667

Umherirrender umherirrender_de...@web.de changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |NEW
   Target Milestone|1.23.0 release  |Future release
Summary|Remove unused rc_cur_time   |Drop unused rc_cur_time
   |field from recentchanges|field from recentchanges
   |table   |table in version 1.24

--- Comment #7 from Umherirrender umherirrender_de...@web.de ---
Field was removed from sql statements.

Milestone is now 1.24

Keeping the field until 1.24 allows easier rollback on version update from 1.22
- 1.23, when something gets wrong with the new version.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l