Fixes to kvutils when used as external workspace (#984)

* Export daml_kvutils_java_proto and add (un)packDamlLogEntryId

Export needed when using kvutils via bazel from another workspace.

* Use 'String' for participant state's LedgerId

Ref.PackageId is the wrong thing, and likely MatchingStringModule
isn't very usable from Java.

ledger/participant-state/kvutils/BUILD.bazel

@@ -15,6 +15,7 @@ da_scala_library(
     visibility = [
         "//visibility:public",
     ],
+    exports = [":daml_kvutils_java_proto"],
     runtime_deps = [],
     deps = [
         ":daml_kvutils_java_proto",

ledger/participant-state/kvutils/src/main/scala/com/daml/ledger/participant/state/kvutils/KeyValueCommitting.scala

@@ -36,6 +36,9 @@ object KeyValueCommitting {
   def packDamlLogEntry(entry: DamlLogEntry): ByteString = entry.toByteString
   def unpackDamlLogEntry(bytes: ByteString): DamlLogEntry = DamlLogEntry.parseFrom(bytes)
 
+  def packDamlLogEntryId(entry: DamlLogEntryId): ByteString = entry.toByteString
+  def unpackDamlLogEntryId(bytes: ByteString): DamlLogEntryId = DamlLogEntryId.parseFrom(bytes)
+
   /** Pretty-printing of the entry identifier. Uses the same hexadecimal encoding as is used
     * for absolute contract identifiers.
     */

ledger/participant-state/src/main/scala/com/daml/ledger/participant/state/v1/package.scala

@@ -58,7 +58,7 @@ import com.digitalasset.daml.lf.value.Value
 package object v1 {
 
   /** Identifier for the ledger, MUST match regexp [a-zA-Z0-9-]. */
-  type LedgerId = Ref.PackageId
+  type LedgerId = String
 
   /** Identifiers for transactions.
     * Currently unrestricted unicode (See issue #398). */