SNOW-1646883: remove disable sql simplifier hack

[sfc-gh-aalam]

1. Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

   Fixes SNOW-1646883

2. Fill out the following pre-review checklist:

   • I am adding a new automated test(s) to verify correctness of my new code
     • If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing
   • I am adding new logging messages
   • I am adding a new telemetry message
   • I am adding new credentials
   • I am adding a new dependency
   • If this is a new feature/behavior, I'm adding the Local Testing parity changes.

3. Please describe how your code solves the related issue.

   Please write a short description of how your code change solves the related issue.

[sfc-gh-aalam]

https://github.com/snowflakedb/snowpark-python/actions/runs/11112251878

[sfc-gh-yzou]

@sfc-gh-aalam is this fixed https://snowflakecomputing.atlassian.net/browse/SNOW-1646883, what is the problem ?

[sfc-gh-aalam]

I don't know. It used to raise error for test described in the ticket. Seems to have fixed now so this change is no longer required

[sfc-gh-yzou]

also @sfc-gh-nkrishna that code seems again violates the architecture in our Snowpark Pandas, that there should be no direct interact with snowpark dataframe, the right way is to provide a static function inside ordered dataframe for range here, and return an ordered dataframe. Then all rest operations should happen on the ordered dataframe. can we get a refactoring task to refact that part?