Re: [PR] [FLINK-38899][runtime-web] Introduce the Rescales/History sub-page for streaming jobs with the adaptive scheduler enabled [flink]

Sun, 05 Apr 2026 19:47:40 -0700 


och5351 commented on PR #27874:
URL: https://github.com/apache/flink/pull/27874#issuecomment-4190083423

   > I checked UI of the PR locally with #27705.
   > 
   > checked:
   > 
   > * [x]  Refresh button related logic and correctness
   > * [x]  Rescales page showing logic
   > * [x]  The table basic information showing logic, like tooltip, table 
headers, columns...,etc.
   > 
   > LGTM +1 on the whole just a few of issues:
   > 
   > * [ ]  highlighted in the Fig.
   > 
   > <img alt="image" width="2000" height="1662" 
src="https://private-user-images.githubusercontent.com/64569824/573576980-0e8acaf9-80cc-46fb-918e-8614f5eb10d4.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NzU0NDAxMjgsIm5iZiI6MTc3NTQzOTgyOCwicGF0aCI6Ii82NDU2OTgyNC81NzM1NzY5ODAtMGU4YWNhZjktODBjYy00NmZiLTkxOGUtODYxNGY1ZWIxMGQ0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNjA0MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjYwNDA2VDAxNDM0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTg2ZmRhODljMTQ3NjQzNTMyZmExYjAzZWE5MTYzOGZlY2YxNTgwZmYyNTg0MTc4OTNiODE3ZGIzODY4NDhiZjYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.Rls11zknecoAvUb_Ak4l9I4BgLm4STPQ2UhOJ9fKNMI";>
 <img alt="image" width="2000" height="640" 
src="https://private-user-images.githubusercontent.com/64569824/573578763-d49e4cbf-c613-4567-8b9f-00b0b63f57a6.png?jwt=e
 
yJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NzU0NDAxMjgsIm5iZiI6MTc3NTQzOTgyOCwicGF0aCI6Ii82NDU2OTgyNC81NzM1Nzg3NjMtZDQ5ZTRjYmYtYzYxMy00NTY3LThiOWYtMDBiMGI2M2Y1N2E2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNjA0MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjYwNDA2VDAxNDM0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ2YTc3ZTVjYjAyY2YxYjk0YjZiMmZjZTQyYmVmZjM4NDZmMDliZjc3MDhhZTBlMDNlMjljOGMyYzAzY2NlZTMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.Ttj12Y85djpsIX8mBEtQhKQFl2CaTuZSsPVYIshVu0w">
   > * [ ]   When I click the Rescales/History from Rescales/Configuration 
page, the  new Rescales/History page did not request the 
`/jobs/:jobid/rescales/history`  interface， We expect that in this case, the 
corresponding API can be requested again.
   > 
   > Thanks @och5351 for the hard work. I conducted some manual tests locally 
and found some minor issues, PTAL next week:)
   
   Hi, @RocMarshal !
   Thank you for review.
   
   Could you please review this again?
   
   ## While waiting
   <img width="2241" height="871" alt="Pasted Graphic 1" 
src="https://github.com/user-attachments/assets/01e0f44d-67e5-4ce8-9383-b68d17170b23";
 />
   
   ## Done
   <img width="2215" height="874" alt="image" 
src="https://github.com/user-attachments/assets/454b96a5-5f7b-4c4c-8fc4-4bb638a376c0";
 />
   
   


-- 
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]

Reply via email to