Airblader commented on pull request #16334: URL: https://github.com/apache/flink/pull/16334#issuecomment-876376561
Thanks for the review @twalthr. I think all your points are valid, but I'd like to address the rest of them in a separate PR. To summarize: * Use the new map config option type in HBase * Convert string constants to enums where possible * (Remove Utils classes afterwards if possible by moving things into the factory directly) -- 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]
