Skip to content

Commit 324414d

Browse files
committed
refactor: Fix some issues
1 parent 048df9a commit 324414d

File tree

3 files changed

+14
-11
lines changed

3 files changed

+14
-11
lines changed

async/async-kafka/src/main/java/org/reactivecommons/async/kafka/KafkaMessage.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@ public String getContentType() {
3030
}
3131
}
3232

33-
public static KafkaMessage fromDelivery(ReceiverRecord<String, byte[]> record) {
34-
return fromDelivery(record, null);
33+
public static KafkaMessage fromDelivery(ReceiverRecord<String, byte[]> receiverRecord) {
34+
return fromDelivery(receiverRecord, null);
3535
}
3636

3737
public static KafkaMessage fromDelivery(ReceiverRecord<String, byte[]> receiverRecord, String type) {

async/async-rabbit/src/main/java/org/reactivecommons/async/rabbit/RabbitDomainEventBus.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313

1414
public class RabbitDomainEventBus implements DomainEventBus {
1515

16+
private static final String EVENT_SEND_FAILURE = "Event send failure: ";
17+
private static final String NOT_IMPLEMENTED_YET = "Not implemented yet";
1618
private final ReactiveMessageSender sender;
1719
private final String exchange;
1820
private final boolean persistentEvents;
@@ -30,35 +32,35 @@ public RabbitDomainEventBus(ReactiveMessageSender sender, String exchange, Broke
3032
@Override
3133
public <T> Mono<Void> emit(DomainEvent<T> event) {
3234
return sender.sendWithConfirm(event, exchange, event.getName(), Collections.emptyMap(), persistentEvents)
33-
.onErrorMap(err -> new RuntimeException("Event send failure: " + event.getName(), err));
35+
.onErrorMap(err -> new RuntimeException(EVENT_SEND_FAILURE + event.getName(), err));
3436
}
3537

3638
@Override
3739
public <T> Publisher<Void> emit(String domain, DomainEvent<T> event) {
38-
throw new UnsupportedOperationException("Not implemented yet");
40+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
3941
}
4042

4143
@Override
4244
public Publisher<Void> emit(CloudEvent cloudEvent) {
4345
return sender.sendWithConfirm(cloudEvent, exchange, cloudEvent.getType(),
4446
Collections.emptyMap(), persistentEvents)
45-
.onErrorMap(err -> new RuntimeException("Event send failure: " + cloudEvent.getType(), err));
47+
.onErrorMap(err -> new RuntimeException(EVENT_SEND_FAILURE + cloudEvent.getType(), err));
4648
}
4749

4850
@Override
4951
public Publisher<Void> emit(String domain, CloudEvent event) {
50-
throw new UnsupportedOperationException("Not implemented yet");
52+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
5153
}
5254

5355
@Override
5456
public Publisher<Void> emit(RawMessage rawEvent) {
5557
return sender.sendWithConfirm(rawEvent, exchange, rawEvent.getType(),
5658
Collections.emptyMap(), persistentEvents)
57-
.onErrorMap(err -> new RuntimeException("Event send failure: " + rawEvent.getType(), err));
59+
.onErrorMap(err -> new RuntimeException(EVENT_SEND_FAILURE + rawEvent.getType(), err));
5860
}
5961

6062
@Override
6163
public Publisher<Void> emit(String domain, RawMessage event) {
62-
throw new UnsupportedOperationException("Not implemented yet");
64+
throw new UnsupportedOperationException(NOT_IMPLEMENTED_YET);
6365
}
6466
}

starters/async-commons-starter/src/main/java/org/reactivecommons/async/starter/senders/GenericDomainEventBus.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515

1616
@RequiredArgsConstructor
1717
public class GenericDomainEventBus implements DomainEventBus {
18+
private static final String DOMAIN_NOT_FOUND = "Domain not found: ";
1819
private final ConcurrentMap<String, DomainEventBus> domainEventBuses;
1920

2021

@@ -27,7 +28,7 @@ public <T> Publisher<Void> emit(DomainEvent<T> event) {
2728
public <T> Publisher<Void> emit(String domain, DomainEvent<T> event) {
2829
DomainEventBus domainEventBus = domainEventBuses.get(domain);
2930
if (domainEventBus == null) {
30-
return Mono.error(() -> new InvalidConfigurationException("Domain not found: " + domain));
31+
return Mono.error(() -> new InvalidConfigurationException(DOMAIN_NOT_FOUND + domain));
3132
}
3233
return domainEventBus.emit(event);
3334
}
@@ -41,7 +42,7 @@ public Publisher<Void> emit(CloudEvent event) {
4142
public Publisher<Void> emit(String domain, CloudEvent event) {
4243
DomainEventBus domainEventBus = domainEventBuses.get(domain);
4344
if (domainEventBus == null) {
44-
return Mono.error(() -> new InvalidConfigurationException("Domain not found: " + domain));
45+
return Mono.error(() -> new InvalidConfigurationException(DOMAIN_NOT_FOUND + domain));
4546
}
4647
return domainEventBus.emit(event);
4748
}
@@ -55,7 +56,7 @@ public Publisher<Void> emit(RawMessage event) {
5556
public Publisher<Void> emit(String domain, RawMessage event) {
5657
DomainEventBus domainEventBus = domainEventBuses.get(domain);
5758
if (domainEventBus == null) {
58-
return Mono.error(() -> new InvalidConfigurationException("Domain not found: " + domain));
59+
return Mono.error(() -> new InvalidConfigurationException(DOMAIN_NOT_FOUND + domain));
5960
}
6061
return domainEventBus.emit(event);
6162
}

0 commit comments

Comments
 (0)