gaborgsomogyi edited a comment on issue #24270: [SPARK-27343][KAFKA][SS]Avoid 
hardcoded for spark-sql-kafka-0-10
URL: https://github.com/apache/spark/pull/24270#issuecomment-483271914
 
 
   Lowercase/Camelcase params is a long standing question (at least for me) in 
this area and because of that I've created a small extract 
[here](https://gist.github.com/gaborgsomogyi/4239c804a3264f680112d366cae46fd1). 
I've started to pick this up in the Kafka part but since the code is working 
because the background implementation is lowercase I've closed the PR.
   
   Basically I'm fine with the actual stand of the PR. The only question where 
do I see the 2 newly added parameters? (this shouldn't be a blocker) I've tried 
the following but it just shows the SQL params:
   ```
   [gaborsomogyi:~/spark] pr/24270+ 7m52s ± ./bin/spark-shell ...
   ...
   scala> spark.sql("SET -v").show(999, false)
   ...
   ```
   

----------------------------------------------------------------
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]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to