Issue 1257:Use datastore to store log info.

Issue 1257:Use datastore to store log info.

[damith, 2014-02-01 11:16:26]

https://codereview.appspot.com/59170045/diff/1/src/main/java/teammates/storag...
File src/main/java/teammates/storage/entity/LogEntry.java (right):

https://codereview.appspot.com/59170045/diff/1/src/main/java/teammates/storag...
src/main/java/teammates/storage/entity/LogEntry.java:61: private long createdAt;
Why not use Date? That's the one we use in other entities.

https://codereview.appspot.com/59170045/diff/1/src/test/java/teammates/test/c...
File src/test/java/teammates/test/cases/logic/FeedbackSessionsLogicTest.java
(right):

https://codereview.appspot.com/59170045/diff/1/src/test/java/teammates/test/c...
src/test/java/teammates/test/cases/logic/FeedbackSessionsLogicTest.java:55:
public class FeedbackSessionsLogicTest extends
BaseComponentUsingTaskQueueTestCase {
Why is this file in the changeset?

[QiaoLiang89, 2014-02-01 13:08:13]

https://codereview.appspot.com/59170045/diff/1/src/main/java/teammates/storag...
File src/main/java/teammates/storage/entity/LogEntry.java (right):

https://codereview.appspot.com/59170045/diff/1/src/main/java/teammates/storag...
src/main/java/teammates/storage/entity/LogEntry.java:61: private long createdAt;
I am trying to keep it consistent with the current loging approach where it was
recorded like this in ActivityLogEntry
"
private long time;
"
and 
"
time = System.currentTimeMillis();
"

https://codereview.appspot.com/59170045/diff/1/src/test/java/teammates/test/c...
File src/test/java/teammates/test/cases/logic/FeedbackSessionsLogicTest.java
(right):

https://codereview.appspot.com/59170045/diff/1/src/test/java/teammates/test/c...
src/test/java/teammates/test/cases/logic/FeedbackSessionsLogicTest.java:55:
public class FeedbackSessionsLogicTest extends
BaseComponentUsingTaskQueueTestCase {
On 2014/02/01 11:16:26, damith wrote:
> Why is this file in the changeset?

Sorry. this shouldn't be here. Messed up indentation

[damith, 2014-02-01 15:11:33]

https://codereview.appspot.com/59170045/diff/1/src/main/java/teammates/storag...
File src/main/java/teammates/storage/entity/LogEntry.java (right):

https://codereview.appspot.com/59170045/diff/1/src/main/java/teammates/storag...
src/main/java/teammates/storage/entity/LogEntry.java:61: private long createdAt;
I see.
It's fine if it is easy to convert that back to a time.
Otherwise go for the Date type which may not need any conversion.

On 2014/02/01 13:08:13, QiaoLiang89 wrote:
> I am trying to keep it consistent with the current loging approach where it
was
> recorded like this in ActivityLogEntry
> "
> private long time;
> "
> and 
> "
> time = System.currentTimeMillis();
> "