[GitHub] [incubator-kvrocks] PragmaTwice commented on a diff in pull request #1274: Fix license issues in rand.h, rocksdb_crc32c.h and batch_debugger.h

["PragmaTwice (via GitHub)" <gi...@apache.org>]

PragmaTwice commented on code in PR #1274:
URL: https://github.com/apache/incubator-kvrocks/pull/1274#discussion_r1113779674


##########
NOTICE:
##########
@@ -223,11 +223,16 @@ It is under the MIT license with an optional exception:
     without including the above copyright and permission notices.
 
 For src/types/geohash.*, src/common/rand.*, src/common/sha1.*, src/storage/scripting.* and
-some functions in src/common/util.cc are modified from Redis.
+some functions in src/common/*_util.* are modified from Redis.
 
 We have added the Redis license notice in each file header, can also see bundled license file
 https://github.com/apache/incubator-kvrocks/blob/unstable/licenses/LICENSE.redis for details.
 
+For src/common/rocksdb_crc32c.* and src/storage/batch_debugger.* are modified from RocksDB.
+
+We have added the RocksDB license notice in each file header, can also see bundled license file
+https://github.com/apache/incubator-kvrocks/blob/unstable/licenses/LICENSE.rocksdb for details.

Review Comment:
   done



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org