abdullah alamoudi has posted comments on this change. Change subject: introduce AlgebraUtils and AlgebraUtils ......................................................................
Patch Set 1: @Yingyi, So I made the new names and I think that I should clarify. when I created AlgebraUtils, I first named it FunctionUtils then found out that we already have that in asterix-lang-common. So I tried to combine them in a single place but dependency issues made it difficult and to avoid confusion, I renamed the new class to AlgebraUtils and then renamed the existing one to LangUtils meaning that this is the util of the lang module and that is is the util of the algebra module. Suggest better names? -- To view, visit https://asterix-gerrit.ics.uci.edu/1069 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic0034bd6ba7f449a03eda48070a8de614608c2dd Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till Westmann <ti...@apache.org> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com> Gerrit-Reviewer: abdullah alamoudi <bamou...@gmail.com> Gerrit-HasComments: No