Skip to content

chore(doc): psql not need the db name #1651

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.

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

Merged
merged 3 commits into from
Apr 7, 2022
Merged

chore(doc): psql not need the db name #1651

merged 3 commits into from
Apr 7, 2022

Conversation

BowenXiao1999
Copy link
Contributor

@BowenXiao1999 BowenXiao1999 commented Apr 7, 2022

What's changed and what's your intention?

It's a known TODO:
https://github.com/singularity-data/risingwave/blob/347406f35d1bb97fe8cd2a4e33cf67b4ff6298c2/src/utils/pgwire/src/pg_protocol.rs#L111-L112

Checklist

  • I have written necessary docs and comments
  • I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Copy link
Contributor

@skyzh skyzh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Can you also help remove all -d dev? They might be available in risedevtool, sqllogictest, and so many other places.

@codecov
Copy link

codecov bot commented Apr 7, 2022

Codecov Report

Merging #1651 (1c85a29) into main (c9c21da) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main    #1651      +/-   ##
============================================
- Coverage     69.79%   69.79%   -0.01%     
  Complexity     2766     2766              
============================================
  Files          1050     1050              
  Lines         92480    92481       +1     
  Branches       1790     1790              
============================================
- Hits          64549    64547       -2     
- Misses        27040    27043       +3     
  Partials        891      891              
Flag Coverage Δ
java 61.01% <ø> (ø)
rust 71.61% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/risedevtool/src/bin/risedev-playground.rs 0.27% <0.00%> (ø)
src/connector/src/filesystem/file_common.rs 79.82% <0.00%> (-0.90%) ⬇️
src/meta/src/stream/meta.rs 31.75% <0.00%> (-0.84%) ⬇️
src/meta/src/barrier/mod.rs 71.71% <0.00%> (-0.29%) ⬇️
src/meta/src/dashboard/mod.rs 0.00% <0.00%> (ø)
src/meta/src/barrier/notifier.rs 45.61% <0.00%> (ø)
src/meta/src/barrier/recovery.rs 0.00% <0.00%> (ø)
...rc/frontend/src/optimizer/property/distribution.rs 85.71% <0.00%> (+1.09%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@BowenXiao1999 BowenXiao1999 force-pushed the bw/refine-doc branch 2 times, most recently from 1ad0468 to 3a53521 Compare April 7, 2022 08:01
@skyzh
Copy link
Contributor

skyzh commented Apr 7, 2022

Will still conflict with #1654 🤪 Maybe you can also help resolve before this PR gets merged.

@skyzh skyzh enabled auto-merge (squash) April 7, 2022 08:29
@skyzh skyzh merged commit 8b6b5ae into main Apr 7, 2022
@skyzh skyzh deleted the bw/refine-doc branch April 7, 2022 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants