dosubot[bot] commented on issue #38468:
URL: https://github.com/apache/superset/issues/38468#issuecomment-4011100691

   <!-- Greeting -->
   Hi @lironsam! I'm [Dosu](https://go.dosu.dev/dosubot) and I’m helping the 
superset team.
   
   <!-- Answer -->
   I can confirm this bug. The issue stems from [PR 
#33542](https://github.com/apache/superset/commit/cf315388f257a24230ce54a78cfce6d1f800fb0c)
 (May 30, 2025), which migrated SQL parsing from sqlparse to sqlglot and 
renamed variables from `statement_num`/`statement_count` to 
`block_num`/`block_count`.
   
   The [French translation 
file](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/translations/fr/LC_MESSAGES/messages.po#L9920-L9922)
 shows the problem clearly:
   ```po
   #, fuzzy, python-format
   msgid "Running block %(block_num)s out of %(block_count)s"
   msgstr "Exécution de l'instruction %(statement_num)s sur %(statement_count)s"
   ```
   
   The `msgid` was updated but the `msgstr` still uses the old variable names, 
causing the `KeyError`.
   
   **This affects more than just French** — at least 10 other languages 
(including Slovenian, Russian, Dutch, Polish, Japanese, Portuguese, German, 
Arabic, Ukrainian, and Persian) have translations referencing the old variable 
names and will experience the same crash.
   
   
   <!-- Dosu Comment Footer -->
   *To reply, just mention [@dosu](https://go.dosu.dev/dosubot).*
   
   ---
   <sup>How did I do? 
[Good](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=hallucination)
 | [Report 
🐛](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/ba6858d9-c755-4fda-b357-e9e4b3367404?feedback_type=other)</sup>
   
   [![Chat with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=github)&nbsp;[![Open
 in 
Cursor](https://dosu.dev/dosu-in-cursor.svg)](https://cursor.com/link/prompt?text=I%20can%20confirm%20this%20bug.%20The%20issue%20stems%20from%20%5BPR%20%2333542%5D%28https%3A//github.com/apache/superset/commit/cf315388f257a24230ce54a78cfce6d1f800fb0c%29%20%28May%2030%2C%202025%29%2C%20which%20migrated%20SQL%20parsing%20from%20sqlparse%20to%20sqlglot%20and%20renamed%20variables%20from%20%60statement_num%60/%60statement_count%60%20to%20%60block_num%60/%60block_count%60.%0A%0AThe%20%5BFrench%20translation%20file%5D%28https%3A//github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/translations/fr/LC_MESSAGES/messages.po%23L9920-L9922%29%20shows%20the%20problem%20clearly%3A%0A%60%60%60po%0A%23%2C%20fuzzy%2C%20python-format%0Amsgid%20%22Running%20block%20%25%28block_num%29s%20out%20of%20%25%28block_count%2
 
9s%22%0Amsgstr%20%22Ex%C3%A9cution%20de%20l%27instruction%20%25%28statement_num%29s%20sur%20%25%28statement_count%29s%22%0A%60%60%60%0A%0AThe%20%60msgid%60%20was%20updated%20but%20the%20%60msgstr%60%20still%20uses%20the%20old%20variable%20names%2C%20causing%20the%20%60KeyError%60.%0A%0A%2A%2AThis%20affects%20more%20than%20just%20French%2A%2A%20%E2%80%94%20at%20least%2010%20other%20languages%20%28including%20Slovenian%2C%20Russian%2C%20Dutch%2C%20Polish%2C%20Japanese%2C%20Portuguese%2C%20German%2C%20Arabic%2C%20Ukrainian%2C%20and%20Persian%29%20have%20translations%20referencing%20the%20old%20variable%20names%20and%20will%20experience%20the%20same%20crash.)&nbsp;[![Join
 
Discord](https://img.shields.io/badge/join-5865F2?logo=discord&logoColor=white&label=)](https://go.dosu.dev/discord-bot)&nbsp;[![Share
 on 
X](https://img.shields.io/badge/X-share-black)](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/issues
 /38468)


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