Hi, When reading the source codes on the master branch, I noticed that some classes and fields names can be renamed for better understanding.
Better to rename these names before releasing because I am not sure whether the generated thrift codes are compatible if we modify them later. e.g., common.thrift: - TRegionLocation to TRegionInfo (seems this is not a "location", though I do not know the defination of "location") - TConfigNodeLocation: * internalEndPoint -> rpcEndPoint - TDataNodeLocation: * externalEndPoint -> clientRpcEndPoint * internalEndPoint -> configNodeRpcEndPoint * dataBlockManagerEndPoint -> dataNodeRpcEndPoint Best, ----------------------------------- Xiangdong Huang School of Software, Tsinghua University
