Ildar Absalyamov has posted comments on this change. Change subject: Merge asterix-experiments to master ......................................................................
Patch Set 2: Ian, just general question, are these experiments supposed to be a part of a build process? I am asking this, since there are a lot of things depending on specific cluster configuration and spatial indexes in this version. Or this change just makes sure that the client in part of the codebase? Another general comment: I found it very helpful to use java8 default methods to implement sort-of-multiple inheritance to configure the exact experiment (thus avoid creating verbose hierarchy of *ExperimentBuilder* classes), you can check that in https://github.com/ty1er/asterix-statistics-experiments/ -- To view, visit https://asterix-gerrit.ics.uci.edu/642 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I79b91e045bd056e62adb6e48795a6625baee0062 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon <[email protected]> Gerrit-Reviewer: Ildar Absalyamov <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Young-Seok Kim <[email protected]> Gerrit-HasComments: No
