plastikat opened a new pull request #467: cqlsh return non-zero status when STDIN CQL fails URL: https://github.com/apache/cassandra/pull/467 I believe currently it can be safely assumed that any `Shell` instance without a TTY (i.e. `Shell.tty == False`) is in a "batch mode". With this in mind, the two following changes were applied. First, while we could just use the `Shell.tty` attribute directly, to be semantically clear and to provide room for future enhancement on what is considered a "batch mode", we will add a respective property instead. Second, with the new `Shell.batch_mode` property at hand, it makes no more sense to check any options implying a "batch mode", it is perfectly enough to just check this property. patch by Jacob Becker a.k.a. plastikat; reviewed by Jordan West for CASSANDRA-15623
---------------------------------------------------------------- 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]
