Repository: fluid-cloudnative/fluid

Issue:
Add comments to  transformResourcesForWorker in pkg\ddc\alluxio\transform_resources.go.
Which function do you want to add comments to?
transformResourcesForWorker in pkg\ddc\alluxio\transform_resources.go.
What comments do you want to add?
// transformResourcesForWorker is responsible for transforming and setting resource limits for the Alluxio Worker component.
// This function updates the resource requirements for the Worker and JobWorker based on the runtime configuration 
// and ensures that memory requests meet the required constraints.
//
// Parameters:
//   - runtime: *datav1alpha1.AlluxioRuntime, the runtime configuration of Alluxio, including resource definitions 
//      for Worker and JobWorker.
//   - value: *Alluxio, the Alluxio runtime instance used to store the transformed resource information.
// 
// Return value:
//   - error: Returns an error if any issue occurs during resource transformation; otherwise, returns nil.

Discussion:

Solution patch:
diff --git a/pkg/ddc/alluxio/transform_resources.go b/pkg/ddc/alluxio/transform_resources.go
index 22f6462b..da765f86 100644
--- a/pkg/ddc/alluxio/transform_resources.go
+++ b/pkg/ddc/alluxio/transform_resources.go
@@ -47,6 +47,17 @@ func (e *AlluxioEngine) transformResourcesForMaster(runtime *datav1alpha1.Alluxi
 
 }
 
+// transformResourcesForWorker is responsible for transforming and setting resource limits for the Alluxio Worker component.
+// This function updates the resource requirements for the Worker and JobWorker based on the runtime configuration 
+// and ensures that memory requests meet the required constraints.
+//
+// Parameters:
+//   - runtime: *datav1alpha1.AlluxioRuntime, the runtime configuration of Alluxio, including resource definitions 
+//      for Worker and JobWorker.
+//   - value: *Alluxio, the Alluxio runtime instance used to store the transformed resource information.
+// 
+// Return value:
+//   - error: Returns an error if any issue occurs during resource transformation; otherwise, returns nil.
 func (e *AlluxioEngine) transformResourcesForWorker(runtime *datav1alpha1.AlluxioRuntime, value *Alluxio) error {
 
 	//for worker



Test patch:
diff --git a/pkg/ddc/alluxio/transform_test.go b/pkg/ddc/alluxio/transform_test.go
index cea65454..4f27550d 100755
--- a/pkg/ddc/alluxio/transform_test.go
+++ b/pkg/ddc/alluxio/transform_test.go
@@ -708,6 +708,16 @@ func TestAlluxioEngine_allocateSinglePort(t *testing.T) {
 	}
 }
 
+// Test function to test the allocatePorts functionality in the AlluxioEngine
+// It sets up the port range for allocation and initializes a port allocator
+// The test cases are structured with different input values and expected results
+// Fields include various configuration values for the Alluxio runtime and test setup
+// Args contain the allocated ports and Alluxio instance to be used in the test cases
+// Test case for setting properties in Alluxio runtime with given allocated ports
+// The test checks that the allocated ports are properly mapped to the expected ports
+// Initialize the AlluxioEngine instance with the test case fields.
+// Call allocatePorts to allocate ports and check for errors.
+// Check if the allocated APIGateway port matches the expected value.
 func TestAlluxioEngine_allocatePorts(t *testing.T) {
 	pr := net.ParsePortRangeOrDie("20000-21000")
 	err := portallocator.SetupRuntimePortAllocator(nil, pr, "bitmap", dummy)