Skip to content
This repository has been archived by the owner on Jun 21, 2019. It is now read-only.

在Windows下连接的时候出错parse goleveldb://data%5Ctidb: invalid URL escape "%5C" #5

Open
insionng opened this issue May 1, 2016 · 7 comments

Comments

@insionng
Copy link

insionng commented May 1, 2016

出错的代码行是这样:
···
return xorm.NewEngine("tidb", "goleveldb://./data/tidb/tidb")
···

@insionng
Copy link
Author

insionng commented May 1, 2016

在Linux下编译的版本没有问题,完全同一份源码,就在Windows下出现这情况~

@insionng insionng changed the title 在Windows是下连接的时候出错parse goleveldb://data%5Ctidb: invalid URL escape "%5C" 在Windows下连接的时候出错parse goleveldb://data%5Ctidb: invalid URL escape "%5C" May 1, 2016
@shenli
Copy link
Contributor

shenli commented May 3, 2016

@insionng Thanks for your feedback!
TiDB supports unix socket now. You can use MySQL driver to connect to TiDB. I will update this repo soon.

@insionng
Copy link
Author

@shenli

呃,请问这个bug修复了没有?

事实上我的项目是比较倾向于选择基于内嵌型的goleveldb,主要是为了替代sqlite,所以其他几个驱动类型不是很合适.

@lunny
Copy link
Member

lunny commented May 14, 2016

这个是不支持windows的问题吗?

@insionng
Copy link
Author

@lunny 不是,应该是不知是tidb还是xorm这边把“/”斜杆给转义了,这在windows下会触发,在linux下没问题。而这个转义的地方我没法知道是tidb还是xorm哪里,否则就自行修改掉了。

@shenli
Copy link
Contributor

shenli commented May 23, 2016

@lunny @insionng 我们正在将 tidb 相关的测试转成用 mysql 的driver,以后就会去掉tidb_driver.go 这个问题也就不存在了。
不过我们发现最新的xorm可能有bug,用tidb会有一些case挂掉,用mysql也遇到了同样的问题,我已经报了issue

@lunny
Copy link
Member

lunny commented May 23, 2016

OK. 最新的PR应该会解决这个问题,我会尽快测试完毕并合并。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants