[Compass-ci] Re: [PATCH compass-ci 1/2] sbin/build-my-info.rb: building my_info from configuration files

...

+def es + ESClient.new(index: 'accounts') end

I think es function is unnecessary.

...

+def store_account_info(my_info) + return if check_account_exist(my_info['my_email'])

+ es.put_source_by_id(my_info['my_email'], my_info)

please use directly here.

because 2 functions -> store_account_info and check_account_exist all use es "ESClient.new(index: 'accounts')", so separate es function. Thanks, Yinsi

Thanks, Liushaofei

...

end

...

+def check_account_exist(email) + return true if es.query_by_id(email)

...

+ return false end

store_account_info(my_info)