hachikuji commented on code in PR #12149:
URL: https://github.com/apache/kafka/pull/12149#discussion_r902836728
##########
clients/src/test/resources/log4j.properties:
##########
@@ -12,7 +12,7 @@
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
-log4j.rootLogger=OFF, stdout
+log4j.rootLogger=ALL, stdout
Review Comment:
Probably added by mistake?
##########
clients/src/main/java/org/apache/kafka/clients/producer/KafkaProducer.java:
##########
@@ -627,6 +627,9 @@ private TransactionManager
configureTransactionState(ProducerConfig config,
* @throws KafkaException if the producer has encountered a previous fatal
error or for any other unexpected error
* @throws TimeoutException if the time taken for initialize the
transaction has surpassed <code>max.block.ms</code>.
* @throws InterruptException if the thread is interrupted while blocked
+ * @throws org.apache.kafka.common.errors.ClusterAuthorizationException
fatal error indicating that the idempotent
Review Comment:
I think we can get rid of this since we have `AuthorizationException` above.
Perhaps we can add the comment about retrying there?
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
