XiaoMi / soar

SQL Optimizer And Rewriter

Apache License 2.0

8.67k stars 1.32k forks source link

ADD: EN of the readme and CI issue #6

Closed AlphaWong closed 5 years ago

AlphaWong commented 5 years ago

Objective

Add the missing part of the EN version readme.
FIx CI issue

martianzhang commented 5 years ago

SOAR now only give the Chinese suggestion. This PR may mislead users that we can give English suggestion now.

SOAR as a command-line tool, we use make daily for daily testing. Travis may help us improve efficiency, we are trying to fix this as soon as possible.

AlphaWong commented 5 years ago

OIC. But the ci yaml file of Travis is not correct. The commit 9bbfa3f will come with a runable yaml. May be u can consider to have to cherry pick for that commit.

As when golang version > 1.9, u need to put double quote in the version number e.g. "1.10"

martianzhang commented 5 years ago

@AlphaWong thanks a lot， suggestion about travis is very useful. Now ci build status come to passed, but there are still many things can do with travis. Thanks for keep following.