fix: [iceberg] order query result deterministically

[hsiang-c]

Which issue does this PR close?

Closes #. #2118

Rationale for this change

• Order the query result by id is not enough to produce deterministic result w/ shuffle. So adding data column to orderBy.
• This PR addresses some of the failures in feat: [iceberg] Enable Comet shuffle in Iceberg diff #2205

What changes are included in this PR?

In Iceberg's 1.8.1.diff, orderBy("id") -> orderBy("id", "data")

How are these changes tested?

Tested passed locally w/ spark.comet.exec.shuffle.mode=jvm and spark.comet.exec.shuffle.mode=native

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.55%. Comparing base (f09f8af) to head (d710388).
⚠️ Report is 400 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2208      +/-   ##
============================================
+ Coverage     56.12%   58.55%   +2.42%     
- Complexity      976     1283     +307     
============================================
  Files           119      143      +24     
  Lines         11743    13216    +1473     
  Branches       2251     2363     +112     
============================================
+ Hits           6591     7738    +1147     
- Misses         4012     4250     +238     
- Partials       1140     1228      +88     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.