Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[yugabyte-client] changes to yugabyte-client tar.gzs #3928

Closed
schoudhury opened this issue Mar 11, 2020 · 3 comments
Closed

[yugabyte-client] changes to yugabyte-client tar.gzs #3928

schoudhury opened this issue Mar 11, 2020 · 3 comments
Assignees
@iSignal
Labels
area/tools

Comments

@schoudhury
Copy link
Contributor

schoudhury commented Mar 11, 2020 •
edited

we recently started generating yugabyte client packages - ysqlsh, cqlsh and redis-cli

https://downloads.yugabyte.com/yugabyte-client-2.1.1.0-linux.tar.gz
https://downloads.yugabyte.com/yugabyte-client-2.1.1.0-darwin.tar.gz

propose the following changes to the above packages:

  1. tar.gz name should be yugabyte-client-2.1-<os>.tar.gz which will signify a single client package for all minor releases across a single major release.
  2. after untar the directory name should be yugabyte-client-2.1. currently it has the same name as the server package yugabyte-2.1.1.0
  3. change cqlsh in the bin directory to ycqlsh (tracked as per [yugabyte-client] rename cqlsh to ycqlsh  #3935)
  4. remove psql as we intend to communicate to users only using ysqlsh
@iSignal iSignal self-assigned this Mar 31, 2020
iSignal added a commit that referenced this issue Apr 4, 2020
@iSignal
[#3928] Remove psql from bin and fix get_clients.sh
46bc746 
Summary: See #3928 for details.

Test Plan: Run get_clients.sh and verify it works as expected

Reviewers: wesley, bogdan, jason

Reviewed By: jason

Subscribers: jenkins-bot

Differential Revision: https://phabricator.dev.yugabyte.com/D8239
@iSignal
Copy link
Contributor

iSignal commented Apr 7, 2020

https://downloads.yugabyte.com/yugabyte-client-2.1-linux.tar.gz
https://downloads.yugabyte.com/yugabyte-client-2.1-dawin.tar.gz
https://downloads.yugabyte.com/get_clients.sh now uses these packages.

psql is not in this package

The cqlsh -> ycqlsh change is more complicated, I have started working on it. Let's track that in the other task separately.

@rkarthik007
Copy link
Collaborator

Super progress!

@iSignal
Copy link
Contributor

iSignal commented May 28, 2020

Closing this for now, all these items should be done. We need to remove cqlsh completely at some point but that's for a later point.

@iSignal iSignal closed this as completed May 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iSignal iSignal

Labels
area/tools
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@schoudhury @iSignal @rkarthik007