Skip to content

IGNITE-17314 .NET: Implement unified exception handling#1074

Merged
ptupitsyn merged 104 commits intoapache:mainfrom
ptupitsyn:ignite-17314
Sep 19, 2022
Merged

IGNITE-17314 .NET: Implement unified exception handling#1074
ptupitsyn merged 104 commits intoapache:mainfrom
ptupitsyn:ignite-17314

Conversation

@ptupitsyn
Copy link
Copy Markdown
Contributor

@ptupitsyn ptupitsyn commented Sep 12, 2022

  • Add all exception groups, codes, and classes from the Java side.
  • When server returns an error, throw corresponding mapped .NET exception type.
  • Use Source Generators to produce C# code for error groups and exception classes. When there is a new error code or exception added on Java side, no changes are required on C# side, only a rebuild.

See generated code here: https://github.com/ptupitsyn/ignite-3/tree/ignite-17314-generated-code-preview/modules/platforms/dotnet/Apache.Ignite/Generated/Apache.Ignite.Internal.Generators

@ptupitsyn ptupitsyn self-assigned this Sep 12, 2022
@ptupitsyn ptupitsyn requested a review from isapego September 13, 2022 05:11
@ptupitsyn ptupitsyn marked this pull request as ready for review September 16, 2022 09:20
@ptupitsyn ptupitsyn merged commit 895502e into apache:main Sep 19, 2022
@ptupitsyn ptupitsyn deleted the ignite-17314 branch September 19, 2022 05:24
lowka pushed a commit to gridgain/apache-ignite-3 that referenced this pull request Mar 18, 2023
* Add all exception groups, codes, and classes from the Java side.
* When server returns an error, throw corresponding mapped .NET exception type.
* Use [Source Generators](https://docs.microsoft.com/en-us/dotnet/csharp/roslyn-sdk/source-generators-overview) to produce C# code for error groups and exception classes. When there is a new error code or exception added on Java side, no changes are required on C# side, only a rebuild.
lowka pushed a commit to gridgain/apache-ignite-3 that referenced this pull request Apr 19, 2023
* Add all exception groups, codes, and classes from the Java side.
* When server returns an error, throw corresponding mapped .NET exception type.
* Use [Source Generators](https://docs.microsoft.com/en-us/dotnet/csharp/roslyn-sdk/source-generators-overview) to produce C# code for error groups and exception classes. When there is a new error code or exception added on Java side, no changes are required on C# side, only a rebuild.
isapego pushed a commit to isapego/ignite-3 that referenced this pull request Dec 26, 2024
Bumps `iceberg` from 1.3.1 to 1.6.1.

Updates `org.apache.iceberg:iceberg-core` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

Updates `org.apache.iceberg:iceberg-data` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

Updates `org.apache.iceberg:iceberg-parquet` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

Updates `org.apache.iceberg:iceberg-aws` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

Updates `org.apache.iceberg:iceberg-snowflake` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

Updates `org.apache.iceberg:iceberg-nessie` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

Updates `org.apache.iceberg:iceberg-hive-metastore` from 1.3.1 to 1.6.1
- [Release notes](https://github.com/apache/iceberg/releases)
- [Commits](apache/iceberg@apache-iceberg-1.3.1...apache-iceberg-1.6.1)

---
updated-dependencies:
- dependency-name: org.apache.iceberg:iceberg-core
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.apache.iceberg:iceberg-data
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.apache.iceberg:iceberg-parquet
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.apache.iceberg:iceberg-aws
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.apache.iceberg:iceberg-snowflake
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.apache.iceberg:iceberg-nessie
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.apache.iceberg:iceberg-hive-metastore
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: Pavel Tupitsyn <ptupitsyn@apache.org>
Co-authored-by: Pavel Tupitsyn <ptupitsyn@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants