xo / usql

Universal command-line interface for SQL databases
MIT License
8.92k stars 351 forks source link

"most" doesn't include apache ignite #75

Closed giftig closed 5 years ago

giftig commented 5 years ago

I installed via brew, and I see it compiling with drivers = "most". Your readme says most is everything other than oracle and ODBC, and you list an apache ignite driver, but when I chck \drivers I see:

Available Drivers:
  avatica [av, phoenix]
  clickhouse [ch]
  cockroachdb (postgres) [cr, cdb, crdb, cockroach]
  cql [ca, scy, scylla, datastax, cassandra]
  firebirdsql [fb, firebird]
  hdb [sa, sap, hana, saphana]
  memsql (mysql) [me]
  mssql [ms, sqlserver]
  mymysql [zm, mymy]
  mysql [my, maria, aurora, mariadb, percona]
  n1ql [n1, couchbase]
  pgx [px]
  postgres [pg, pgsql, postgresql]
  presto [pr, prs, prestos, prestodb, prestodbs]
  sqlite3 [sq, file, sqlite]
  tidb (mysql) [ti]
  vitess (mysql) [vt]
  voltdb [vo, vdb, volt]

no apache ignite. Not sure if this is an omission from your readme and deliberately excluded from "most" as it's a less common use case, or if it's been accidentally excluded from "most".

kenshaw commented 5 years ago

It's definitely part of the most tag. Can you give me some more information? Perhaps you need to update the brew tap?

giftig commented 5 years ago

@kenshaw I installed the brew tap fresh right before installing, so I assume I'd have the latest version. I don't have my mac at hand right now but if you let me know what other information will help I can provide it when I do.

I installed it with the most basic example on the readme, i.e.

brew tap xo/xo
brew install usql
kenshaw commented 5 years ago

@giftig ugh -- sorry for the massively edited comment. The apache ignite driver for Go is not maintained.

giftig commented 5 years ago

No worries, mercifully I didn't have to look at apache ignite for long.