Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2024-01-15 Thread via GitHub
gaborgsomogyi merged PR #23930: URL: https://github.com/apache/flink/pull/23930 -- 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-unsubscr...@flink

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2024-01-14 Thread via GitHub
gyfora commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1891493012 Looks good @gaborgsomogyi 🚢 -- 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 co

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2024-01-13 Thread via GitHub
gaborgsomogyi commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1890425879 Seems rebase did the job. Any comments, @gyfora ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2024-01-12 Thread via GitHub
gaborgsomogyi commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1889300134 Unrelated error again: ``` Jan 12 13:24:31 mypy: can't read file '/__w/3/s/flink-python/dev/.conda/lib/python3.10/site-packages//google': No such file or directory ``` R

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2024-01-11 Thread via GitHub
gaborgsomogyi commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1888577653 The change looks good, but timeout happened. -- 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 t

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2024-01-11 Thread via GitHub
gaborgsomogyi commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1888576964 @flinkbot run azure -- 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 comm

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2023-12-14 Thread via GitHub
gaborgsomogyi commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1856093109 cc @gyfora -- 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

Re: [PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2023-12-14 Thread via GitHub
flinkbot commented on PR #23930: URL: https://github.com/apache/flink/pull/23930#issuecomment-1856059558 ## CI report: * cdef4fd4b4b21dad5c1ae52b9fd2a17ddb101a7f UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-33268][rest] Skip unknown fields in REST response deserialization [flink]

2023-12-14 Thread via GitHub
gaborgsomogyi opened a new pull request, #23930: URL: https://github.com/apache/flink/pull/23930 ## What is the purpose of the change At the moment Flink is throwing exception on unknown fields when parsing REST responses. An example for such a class is `JobDetailsInfo` but this appl