dyninst: log more info about SymDB uploads

Author: andreimatei

pkg/dyninst/module/symdb.go

@@ -575,6 +575,7 @@ func (m *symdbManager) performUpload(
 
 	log.Infof("SymDB: uploading symbols for process %v (service: %s, version: %s, executable: %s)",
 		procID.pid, procID.service, procID.version, executablePath)
+	startTime := time.Now()
 	it, err := symdb.PackagesIterator(
 		executablePath,
 		m.objectLoader,
@@ -591,7 +592,8 @@ func (m *symdbManager) performUpload(
 	uploadBuffer := make([]uploader.Scope, 0, 100)
 	bufferFuncs := 0
 	uploadID := uuid.New()
-	// Flush every so ofter in order to not store too many scopes in memory.
+	var totalPackages, totalFuncs, numChunks int
+	// Flush every so often in order to not store too many scopes in memory.
 	maybeFlush := func(final bool) error {
 		if ctx.Err() != nil {
 			return context.Cause(ctx)
@@ -612,6 +614,7 @@ func (m *symdbManager) performUpload(
 			if err != nil {
 				return fmt.Errorf("upload failed: %w", err)
 			}
+			numChunks++
 			uploadBuffer = uploadBuffer[:0]
 			bufferFuncs = 0
 		}
@@ -629,13 +632,18 @@ func (m *symdbManager) performUpload(
 
 		scope := uploader.ConvertPackageToScope(pkg.Package, version.AgentVersion)
 		uploadBuffer = append(uploadBuffer, scope)
+		totalPackages++
+		totalFuncs += pkg.Stats().NumFunctions
 		bufferFuncs += pkg.Stats().NumFunctions
 		if err := maybeFlush(pkg.Final); err != nil {
 			return err
 		}
 	}
 
-	log.Infof("SymDB: Successfully uploaded symbols for process %v (service: %s, version: %s, executable: %s)",
-		procID.pid, procID.service, procID.version, executablePath)
+	log.Infof("SymDB: Successfully uploaded symbols for process %v "+
+		"(service: %s, version: %s, executable: %s):"+
+		" %d packages, %d functions, %d chunks in %v",
+		procID.pid, procID.service, procID.version, executablePath,
+		totalPackages, totalFuncs, numChunks, time.Since(startTime))
 	return nil
 }