RocMarshal commented on PR #235: URL: https://github.com/apache/flink-connector-aws/pull/235#issuecomment-3932810395
Hi, @ferenc-csaky Please let me have a try on clarifying the reason to drive the PR. as described in https://github.com/apache/flink/pull/27543#issuecomment-3860763538 > I do not see how or why this is an issue. > > * This was like this since since ever and worked. > * If I paste the link to the browser in the updated format, it will get redirected to `/#` anyways, so that's where the actual resource is. > * If I search for the `/#` syntax in the core Flink repo documentation folder, there are tons of other links that uses this same syntax. > > And the [doc section](https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=72542&view=results) of the latest run for your [PR](https://github.com/apache/flink/pull/27543) succeeded, which also states that this is correct as is. > > Hence, I'm -1 for merging this, as this does not fix anything. Thanks @ferenc-csaky for the quick response. Sounds good to me. And I'll re-check the cause of the issue mentioned in previous PR. :) -- 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]
