asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jianfeng Jia (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Not suppose to MERGE, for discussion purpose The following c...
Date Thu, 01 Sep 2016 03:35:46 GMT
Jianfeng Jia has posted comments on this change.

Change subject: Not suppose to MERGE, for discussion purpose The following commits from your
working branch will be included:
......................................................................


Patch Set 1:

@Yingyi, I think there maybe the issue of the Math.pow() which gives an approximate result.
Here I tested the Int number from 1 to Int.Max. The result is as expected, but this test gonna
take looong to execute. Any suggestions?

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1135
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id6d437ccff51adb236f4e69c78761a2c336ae71a
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Jianfeng Jia <jianfeng.jia@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Jianfeng Jia <jianfeng.jia@gmail.com>
Gerrit-Reviewer: Yingyi Bu <buyingyi@gmail.com>
Gerrit-HasComments: No

Mime
View raw message