rmahindra123 commented on pull request #3117: URL: https://github.com/apache/hudi/pull/3117#issuecomment-870852282
> I see we have more test coverage in TestExternalSpillableMap. Can we parametrize the tests that are applicable and run those for both type of spillable maps. > > In general, do we have tests around diff values for maxInMemorySizeInBytes for external spillable map. If not, do you think we can add them while we are at this. I had tests for TestExternalSpillableMap in the stacked diff, anyway added it here. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
