jextract: Preserve underlying exception when async closure completes exceptionally

Đang mở
#915 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
65/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java, swift
Lĩnh vực
devtools

Hướng nghiên cứu

Bắt đầu từ Swift thunk bắt các ngoại lệ từ một Java async closure và theo dõi cách exceptional completion được truyền đến Java future. Tái hiện trường hợp RuntimeException từ issue và xác minh rằng ngoại lệ bên dưới được giữ lại làm cause thay vì bị thay thế bằng một Exception chung chung.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

When a Java async closure completes exceptionally (e.g. with RuntimeException), the error is currently caught in the Swift thunk and completed exceptionally on the Java future as a generic Exception:

Future<Void> future = MySwiftLibrary.asyncScheduleThrowing(() -> {
    CompletableFuture<Void> failed = new CompletableFuture<>();
    failed.completeExceptionally(new RuntimeException("Java async failed"));
    return failed;
});

assertThrows(ExecutionException.class, future::get);

Optimally, we should unwrap the ExecutionException in the Swift thunk and rethrow/preserve the underlying RuntimeException as the cause rather than wrapping it in a generic Exception.

Ngôn ngữ chính
Swift
Star
1.3k
Fork
127
Merge trung bình
2 ngày 14 giờ
Pull request đã merge (30 ngày)
5

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của swiftlang/swift-java

Tất cả issue của swiftlang/swift-java

Issue tương tự

Thêm issue về Swift

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.