nus-oss-test / testrepo4

TEAMMATES system is online at
http://teammatesv4.appspot.com
0 stars 0 forks source link

Use datastore to store log info. #173

Open damithc opened 10 years ago

damithc commented 10 years ago

From QiaoLian...@gmail.com on October 22, 2013 16:44:49

use datastore to store log info with proper format, instead of tokenising and parsing a logline from google

Original issue: http://code.google.com/p/teammatespes/issues/detail?id=1257

damithc commented 10 years ago

From QiaoLian...@gmail.com on October 22, 2013 06:30:51

Status: Accepted

damithc commented 10 years ago

From QiaoLian...@gmail.com on January 28, 2014 21:28:26

https://codereview.appspot.com/54680044/ Am I suppose to discard those binary files? They comes in after the merge.

Status: ReadyForReview

damithc commented 10 years ago

From dam...@gmail.com on January 28, 2014 22:24:23

Something wrong with how you merged :-p

damithc commented 10 years ago

From QiaoLian...@gmail.com on February 01, 2014 02:02:38

I think I messed up .hgignore ... And that's why lots of garbage ends up in the commit >.< http://codereview.appspot.com/59170045

damithc commented 10 years ago

From dam...@gmail.com on February 01, 2014 03:17:06

Waiting for clarifications.

Status: ChangesRequested

damithc commented 10 years ago

From dam...@gmail.com on February 01, 2014 07:14:20

Try to leave out the files not intended to change. Remember to get dev green before uploading.

Status: ReadyToMerge

damithc commented 10 years ago

From dam...@gmail.com on February 20, 2014 23:11:04

Qiao Liang, you are supposed to merge these yourself. Instructions in dev man.

damithc commented 10 years ago

From dam...@gmail.com on March 11, 2014 21:05:31

The performance cost is too high. Increases test time by about 40%. We may have to get cron jobs and tasks queues involved.

Status: ChangesRequested

damithc commented 10 years ago

From QiaoLian...@gmail.com on March 25, 2014 23:39:51

http://codereview.appspot.com/80490043

Status: ReadyForReview

damithc commented 10 years ago

From dam...@gmail.com on March 28, 2014 20:10:52

This may not be a good time to merge this. Shelving this to be merged in a future date, probably sometime in the summer.

Status: Shelved