Added support to process empty records

[vmalaviya-splunk]

No description provided.

[fantavlik]

I have some additional suggestions - we talked about adding tests, see below.

More intuitive name

I think allow_empty_list probably doesn't mean anything to our users so maybe something more along the lines of allow_empty_chunks or allow_empty_records

Generating commands

I was also thinking about generating commands, they have their own implementation for _execute_chunk_v2 that obviously assumes that no events are present because that's how generating commands work. So we should probably enforce that by not supplying to parameter in the process method for generating commands, so adding this to https://github.com/splunk/splunk-sdk-python/blob/master/splunklib/searchcommands/generating_command.py (add adding import sys:

    def process(self, argv=sys.argv, ifile=sys.stdin, ofile=sys.stdout):
        """ Process data.

        :param argv: Command line arguments.
        :type argv: list or tuple

        :param ifile: Input data file.
        :type ifile: file

        :param ofile: Output data file.
        :type ofile: file

        :return: :const:`None`
        :rtype: NoneType

        """
        # Force allow_empty_list to be true since generating commands assume an empty input
        return self.process(argv=argv, ifile=ifile, ofile=ofile, allow_empty_list=True)

Allow empty by default

I was also thinking about actually setting allow_empty_list to True by default for all commands since that seems to match our previous behavior.

Let me know what you think.

• Change name of flag to something more intuitive like allow_empty_chunks or `allow_empty_records
• Add tests for allow_empty_list in https://github.com/splunk/splunk-sdk-python/blob/master/tests/searchcommands/test_search_command.py
• Change logic to enforce that generating commands always allow empty input records
• Change to allow empty records by default instead of throwing an error (True instead of False)

[fantavlik]

Add a docstring below for the new arg

[fantavlik]

Slightly more descriptive:
Allow empty input records for the command, if False an Error will be returned if empty chunk body is encountered when read

[fantavlik]

Added suggestion for a unit test and code comment but overall this looks good - feel free to merge when those are added

[fantavlik]

Thanks for all the updates, looks great 🚀