sanel commented on a change in pull request #2107:
URL: https://github.com/apache/drill/pull/2107#discussion_r504936707
##########
File path: contrib/native/client/src/clientlib/errmsgs.hpp
##########
@@ -59,7 +59,8 @@ namespace Drill{
#define ERR_CONN_SSL_CN DRILL_ERR_START+27
#define ERR_CONN_SSL_CERTVERIFY DRILL_ERR_START+28
#define ERR_CONN_SSL_PROTOVER DRILL_ERR_START+29
-#define ERR_CONN_MAX DRILL_ERR_START+29
+#define ERR_CONN_SSL_SNI DRILL_ERR_START+30
+#define ERR_CONN_MAX DRILL_ERR_START+30
Review comment:
In that case I'd just add comment for the last entry. Easy to overlook.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
