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.

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

Couple of copy contructor and/or assignment operator missing. #6

Closed
wants to merge 1 commit into from

Conversation

Carmot
Copy link

@Carmot Carmot commented Oct 30, 2017

@qinzuoyan
Copy link
Contributor

Thanks for you propose. Because the RocksDB project is forked from facebook/rocksdb, so I think you'd better send pull request to facebook/rocksdb firstly.

@qinzuoyan qinzuoyan closed this Mar 13, 2018
acelyc111 pushed a commit that referenced this pull request Apr 26, 2022
cauchy1988 pushed a commit to cauchy1988/incubator-pegasus that referenced this pull request May 9, 2022
cauchy1988 pushed a commit to cauchy1988/incubator-pegasus that referenced this pull request May 12, 2022
* add more API docstring
acelyc111 pushed a commit that referenced this pull request May 26, 2022
bugfix: revert pr#3 and fix connections being closed during TableConn…
acelyc111 pushed a commit that referenced this pull request May 26, 2022
fix meta session re-connection bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants