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

distsqlrun: forward-port regression test for topk panic #25240

Merged
merged 1 commit into from
May 2, 2018

Conversation

jordanlewis
Copy link
Member

Release note: None

@jordanlewis jordanlewis requested review from rjnn and a team May 2, 2018 15:51
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rjnn
Copy link
Contributor

rjnn commented May 2, 2018

:lgtm:


Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@jordanlewis
Copy link
Member Author

bors r+

craig bot pushed a commit that referenced this pull request May 2, 2018
21580: libroach: encrypt data at rest r=mberhault a=mberhault

Part of encryption-at-rest (see #19783)

Quick overview:
* file registry records encryption settings for encrypted files
* an encrypted env has a "env level" (plain, store, data) and a key
manager as a source
* the data key manager uses an encrypted env with the store key manager as
key source
* rocksdb uses an encrypted env with the data key manager as key source

Release note: none

25240: distsqlrun: forward-port regression test for topk panic r=jordanlewis a=jordanlewis

Release note: None

Co-authored-by: marc <[email protected]>
Co-authored-by: Jordan Lewis <[email protected]>
@craig
Copy link
Contributor

craig bot commented May 2, 2018

Build succeeded

@craig craig bot merged commit 07e25f8 into cockroachdb:master May 2, 2018
@jordanlewis jordanlewis deleted the regression-test branch May 21, 2018 03:00
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.

3 participants