Skip to content

Commit 928087d

Browse files
authored
Export functions from Key Registry (dgraph-io#1561)
Export some of the functions from key registry which are to be used by dgraph for the encryption of raftwal.
1 parent ea932a8 commit 928087d

File tree

6 files changed

+19
-19
lines changed

6 files changed

+19
-19
lines changed

db.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1000,7 +1000,7 @@ func (db *DB) handleFlushTask(ft flushTask) error {
10001000
return nil
10011001
}
10021002

1003-
dk, err := db.registry.latestDataKey()
1003+
dk, err := db.registry.LatestDataKey()
10041004
if err != nil {
10051005
return y.Wrapf(err, "failed to get datakey in db.handleFlushTask")
10061006
}

key_registry.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -294,8 +294,8 @@ func WriteKeyRegistry(reg *KeyRegistry, opt KeyRegistryOptions) error {
294294
return syncDir(opt.Dir)
295295
}
296296

297-
// dataKey returns datakey of the given key id.
298-
func (kr *KeyRegistry) dataKey(id uint64) (*pb.DataKey, error) {
297+
// DataKey returns datakey of the given key id.
298+
func (kr *KeyRegistry) DataKey(id uint64) (*pb.DataKey, error) {
299299
kr.RLock()
300300
defer kr.RUnlock()
301301
if id == 0 {
@@ -309,10 +309,10 @@ func (kr *KeyRegistry) dataKey(id uint64) (*pb.DataKey, error) {
309309
return dk, nil
310310
}
311311

312-
// latestDataKey will give you the latest generated datakey based on the rotation
312+
// LatestDataKey will give you the latest generated datakey based on the rotation
313313
// period. If the last generated datakey lifetime exceeds the rotation period.
314314
// It'll create new datakey.
315-
func (kr *KeyRegistry) latestDataKey() (*pb.DataKey, error) {
315+
func (kr *KeyRegistry) LatestDataKey() (*pb.DataKey, error) {
316316
if len(kr.opt.EncryptionKey) == 0 {
317317
// nil is for no encryption.
318318
return nil, nil

key_registry_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,12 @@ func TestBuildRegistry(t *testing.T) {
4242

4343
kr, err := OpenKeyRegistry(opt)
4444
require.NoError(t, err)
45-
dk, err := kr.latestDataKey()
45+
dk, err := kr.LatestDataKey()
4646
require.NoError(t, err)
4747
// We're resetting the last created timestamp. So, it creates
4848
// new datakey.
4949
kr.lastCreated = 0
50-
dk1, err := kr.latestDataKey()
50+
dk1, err := kr.LatestDataKey()
5151
// We generated two key. So, checking the length.
5252
require.Equal(t, 2, len(kr.dataKeys))
5353
require.NoError(t, err)
@@ -72,12 +72,12 @@ func TestRewriteRegistry(t *testing.T) {
7272
opt := getRegistryTestOptions(dir, encryptionKey)
7373
kr, err := OpenKeyRegistry(opt)
7474
require.NoError(t, err)
75-
_, err = kr.latestDataKey()
75+
_, err = kr.LatestDataKey()
7676
require.NoError(t, err)
7777
// We're resetting the last created timestamp. So, it creates
7878
// new datakey.
7979
kr.lastCreated = 0
80-
_, err = kr.latestDataKey()
80+
_, err = kr.LatestDataKey()
8181
require.NoError(t, err)
8282
require.NoError(t, kr.Close())
8383
delete(kr.dataKeys, 1)
@@ -123,14 +123,14 @@ func TestEncryptionAndDecryption(t *testing.T) {
123123
opt := getRegistryTestOptions(dir, encryptionKey)
124124
kr, err := OpenKeyRegistry(opt)
125125
require.NoError(t, err)
126-
dk, err := kr.latestDataKey()
126+
dk, err := kr.LatestDataKey()
127127
require.NoError(t, err)
128128
require.NoError(t, kr.Close())
129129
// Checking the correctness of the datakey after closing and
130130
// opening the key registry.
131131
kr, err = OpenKeyRegistry(opt)
132132
require.NoError(t, err)
133-
dk1, err := kr.dataKey(dk.GetKeyId())
133+
dk1, err := kr.DataKey(dk.GetKeyId())
134134
require.NoError(t, err)
135135
require.Equal(t, dk.Data, dk1.Data)
136136
require.NoError(t, kr.Close())
@@ -146,12 +146,12 @@ func TestKeyRegistryInMemory(t *testing.T) {
146146

147147
kr, err := OpenKeyRegistry(opt)
148148
require.NoError(t, err)
149-
_, err = kr.latestDataKey()
149+
_, err = kr.LatestDataKey()
150150
require.NoError(t, err)
151151
// We're resetting the last created timestamp. So, it creates
152152
// new datakey.
153153
kr.lastCreated = 0
154-
_, err = kr.latestDataKey()
154+
_, err = kr.LatestDataKey()
155155
// We generated two key. So, checking the length.
156156
require.Equal(t, 2, len(kr.dataKeys))
157157
require.NoError(t, err)

levels.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ func newLevelsController(db *DB, mf *Manifest) (*levelsController, error) {
137137
throttle.Done(rerr)
138138
atomic.AddInt32(&numOpened, 1)
139139
}()
140-
dk, err := db.registry.dataKey(tf.KeyID)
140+
dk, err := db.registry.DataKey(tf.KeyID)
141141
if err != nil {
142142
rerr = y.Wrapf(err, "Error while reading datakey")
143143
return
@@ -552,7 +552,7 @@ nextTable:
552552
inflightBuilders := y.NewThrottle(5)
553553
for it.Valid() {
554554
timeStart := time.Now()
555-
dk, err := s.kv.registry.latestDataKey()
555+
dk, err := s.kv.registry.LatestDataKey()
556556
if err != nil {
557557
return nil, nil,
558558
y.Wrapf(err, "Error while retrieving datakey in levelsController.compactBuildTables")

memtable.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -565,7 +565,7 @@ func (lf *logFile) open(path string, flags int, opt Options) error {
565565
"Unable to copy from %s, size %d", path, lf.size)
566566
keyID := binary.BigEndian.Uint64(buf[:8])
567567
// retrieve datakey.
568-
if dk, err := lf.registry.dataKey(keyID); err != nil {
568+
if dk, err := lf.registry.DataKey(keyID); err != nil {
569569
return y.Wrapf(err, "While opening vlog file %d", lf.fid)
570570
} else {
571571
lf.dataKey = dk
@@ -587,7 +587,7 @@ func (lf *logFile) bootstrap() error {
587587

588588
// generate data key for the log file.
589589
var dk *pb.DataKey
590-
if dk, err = lf.registry.latestDataKey(); err != nil {
590+
if dk, err = lf.registry.LatestDataKey(); err != nil {
591591
return y.Wrapf(err, "Error while retrieving datakey in logFile.bootstarp")
592592
}
593593
lf.dataKey = dk

stream_writer.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ type sortedWriter struct {
248248
}
249249

250250
func (sw *StreamWriter) newWriter(streamID uint32) (*sortedWriter, error) {
251-
dk, err := sw.db.registry.latestDataKey()
251+
dk, err := sw.db.registry.LatestDataKey()
252252
if err != nil {
253253
return nil, err
254254
}
@@ -351,7 +351,7 @@ func (w *sortedWriter) send(done bool) error {
351351
return nil
352352
}
353353

354-
dk, err := w.db.registry.latestDataKey()
354+
dk, err := w.db.registry.LatestDataKey()
355355
if err != nil {
356356
return y.Wrapf(err, "Error while retriving datakey in sortedWriter.send")
357357
}

0 commit comments

Comments
 (0)