You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/06/25 15:05:16 UTC
[GitHub] [flink-kubernetes-operator] morhidi opened a new pull request, #280: [FLINK-28008] Can not get secondary resource from context after operator restart
morhidi opened a new pull request, #280:
URL: https://github.com/apache/flink-kubernetes-operator/pull/280
- bumping JOSDK version v3.0.2 -> v3.0.3
- add primaryToSecondary mapper for session jobs
- removing the workaround for setting namespaces (not needed anymore from JOSDK v3.0.3+)
--
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: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [flink-kubernetes-operator] morhidi commented on pull request #280: [FLINK-28008] Can not get secondary resource from context after operator restart
Posted by GitBox <gi...@apache.org>.
morhidi commented on PR #280:
URL: https://github.com/apache/flink-kubernetes-operator/pull/280#issuecomment-1166319359
cc @Aitozi
--
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: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #280: [FLINK-28008] Can not get secondary resource from context after operator restart
Posted by GitBox <gi...@apache.org>.
gyfora commented on code in PR #280:
URL: https://github.com/apache/flink-kubernetes-operator/pull/280#discussion_r907108951
##########
flink-kubernetes-operator/src/main/resources/log4j2.properties:
##########
@@ -24,3 +24,6 @@ appender.console.name = ConsoleAppender
appender.console.type = CONSOLE
appender.console.layout.type = PatternLayout
appender.console.layout.pattern = %style{%d}{yellow} %style{%-30c{1.}}{cyan} %highlight{[%-5level] [%X{resource.namespace}.%X{resource.name}] %msg%n%throwable}
+
+logger.conf.name = org.apache.flink.configuration.GlobalConfiguration
+logger.conf.level = WARN
Review Comment:
makes sense :)
--
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: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [flink-kubernetes-operator] gyfora merged pull request #280: [FLINK-28008] Can not get secondary resource from context after operator restart
Posted by GitBox <gi...@apache.org>.
gyfora merged PR #280:
URL: https://github.com/apache/flink-kubernetes-operator/pull/280
--
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: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #280: [FLINK-28008] Can not get secondary resource from context after operator restart
Posted by GitBox <gi...@apache.org>.
gyfora commented on code in PR #280:
URL: https://github.com/apache/flink-kubernetes-operator/pull/280#discussion_r906791270
##########
flink-kubernetes-operator/src/main/resources/log4j2.properties:
##########
@@ -24,3 +24,6 @@ appender.console.name = ConsoleAppender
appender.console.type = CONSOLE
appender.console.layout.type = PatternLayout
appender.console.layout.pattern = %style{%d}{yellow} %style{%-30c{1.}}{cyan} %highlight{[%-5level] [%X{resource.namespace}.%X{resource.name}] %msg%n%throwable}
+
+logger.conf.name = org.apache.flink.configuration.GlobalConfiguration
+logger.conf.level = WARN
Review Comment:
Why did you change this? Isn't it helpful to see what exact config is loaded by the operator?
--
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: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [flink-kubernetes-operator] morhidi commented on a diff in pull request #280: [FLINK-28008] Can not get secondary resource from context after operator restart
Posted by GitBox <gi...@apache.org>.
morhidi commented on code in PR #280:
URL: https://github.com/apache/flink-kubernetes-operator/pull/280#discussion_r907100122
##########
flink-kubernetes-operator/src/main/resources/log4j2.properties:
##########
@@ -24,3 +24,6 @@ appender.console.name = ConsoleAppender
appender.console.type = CONSOLE
appender.console.layout.type = PatternLayout
appender.console.layout.pattern = %style{%d}{yellow} %style{%-30c{1.}}{cyan} %highlight{[%-5level] [%X{resource.namespace}.%X{resource.name}] %msg%n%throwable}
+
+logger.conf.name = org.apache.flink.configuration.GlobalConfiguration
+logger.conf.level = WARN
Review Comment:
It's annoyingly verbose atm:) I'll add a one-liner instead.
--
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: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org