[Bug report] In IcebergRequestContext.java httpServletRequest is deprecated but still used

[justinmclean]

Version

main branch

Describe what's wrong

In IcebergRequestContext.java httpServletRequest is deprecated but still used in the constructor.

  this.httpServletRequest = httpRequest;

A comment states "To keep compatibility with old IcebergRequestContext, will remove in new release."

Error message and/or stacktrace

Should this be fixed in this release?

How to reproduce

Current main (0.7)

Additional context

No response

[FANNG1]

This is deprecated in 0.8, so may remove it in 0.10 or latter version .

[justinmclean]

If it deprecated in 0.8, why are we still using it?

[FANNG1]

This is the event API published to user, we should keep compatibility to make old code works.