iMashtak commented on code in PR #12625:
URL: https://github.com/apache/camel/pull/12625#discussion_r1439721733


##########
components/camel-netty/src/test/java/org/apache/camel/component/netty/NettyTCPSyncUDSTest.java:
##########
@@ -19,16 +19,14 @@
 import org.apache.camel.Exchange;
 import org.apache.camel.Processor;
 import org.apache.camel.builder.RouteBuilder;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 /**
- * Add {@code <classifier>linux-x86_64</classifier>} to 
io.netty:netty-transport-native-epoll dependency to make this
- * test work
+ *
+ * @see <a href="https://netty.io/wiki/native-transports.html";>about netty 
native transport</a>
  */
-@Disabled("Requires native library to load, can be run manually")

Review Comment:
   Not only on Mac - on Linux too. User must include library with bindings, for 
example:
   ```xml
   <dependency>
     <groupId>io.netty</groupId>
     <artifactId>netty-transport-native-epoll</artifactId>
     <version>${netty-version}</version>
     <classifier>linux-x86_64</classifier>
   </dependency>
   ```
   
   I will make soon new doc-section about native transports for netty - i think 
that will be more pretty then any setting description



-- 
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: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to