Re: [PR] [SPARK-46938][BUILD][CORE][SQL][UI] Migrate from Jetty 10 to Jetty 11 [spark]

Mon, 19 Feb 2024 23:03:15 -0800 


HiuKwok commented on PR #45154:
URL: https://github.com/apache/spark/pull/45154#issuecomment-1953596603

   > BTW, I realized that we need to make this PR less intrusive by reducing 
the surface of change. We had better introduce a layer inside Spark. Not only 
for this but also for the migration from Jetty 11 to Jerry 12.
   > 
   > ![Screenshot 2024-02-19 at 10 46 
52](https://private-user-images.githubusercontent.com/9700541/306050790-ce5922b2-3668-45dc-931b-7f0133aab2e8.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDg0MTA5MDAsIm5iZiI6MTcwODQxMDYwMCwicGF0aCI6Ii85NzAwNTQxLzMwNjA1MDc5MC1jZTU5MjJiMi0zNjY4LTQ1ZGMtOTMxYi03ZjAxMzNhYWIyZTgucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDIyMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDAyMjBUMDYzMDAwWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDFiM2VmZjUxY2E1NDFjYjE1MDcwMzNiYjQxZTViM2NkYWQyNTA1MmIzZmVhZTNmZGI2OTBhZTEzYjNjYmViYiZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.Cya0G3D2oAqUommq0CjTyGl-xEwnToiSoBZYSIDy3G4)
   > 
   > I made a PR for that.
   > 
   > * [[SPARK-47092][CORE][SQL][K8S] Add `getUriBuilder` to `o.a.s.u.Utils` 
and use it #45168](https://github.com/apache/spark/pull/45168)
   
   Sure, I review the existing code change and check whether how can we 
centralise the servlet reference, mostly likely this will be another MR prior 
to this upgrade. 


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to